Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

feat: transaction 모듈 수정 및 event 입금기한 추가 #24

Merged
merged 7 commits into from
May 4, 2024
Merged

feat: transaction 모듈 수정 및 event 입금기한 추가 #24

merged 7 commits into from
May 4, 2024

Conversation

ssoxong
Copy link
Contributor

@ssoxong ssoxong commented May 1, 2024

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

업데이트 요약

  • 엑셀 파싱 라이브러리 exceljs -> xlsx-populate 변경
  • 거래내역 업로드 시 비밀번호도 같이 받기
  • event 입금기한으로 거래내역 알려주는 controller 추가
  • event module에 입금기한 추가

해결한 문제

  • xlsx-populate가 js 기반이라 자꾸 워닝 뜨길래 dependencies에 추가
    "@types/xlsx-populate": "github:JanLoebel/types-xlsx-populate"
    • ts용으로 사용가능하게 해줌

미해결 문제

테스트

  • 유닛 테스트
  • 빌드 테스트 (통합 테스트)
  • 기타 유효성 테스트

수정 사항 진단

  • 코드가 이 프로젝트의 스타일 지침을 따릅니다.
  • 코드에 대한 자체 리뷰를 수행했습니다.
  • 변경 내역에 대해 주석을 작성했습니다.
  • 해당 PR에 대한 문서를 변경했습니다.
  • 기능, 정책, 수정 사항, 이슈에 대한 테스트 코드를 추가했습니다.
  • 변경 내용이 로컬 개발환경에서의 테스트를 통과했습니다.
  • 모든 종속 변경 사항이 병합되어 다운스트림 모듈에 게시되었습니다.

@ssoxong ssoxong changed the title feat/transaction feat: transaction 모듈 수정 및 event 입금기한 추가 May 1, 2024
w8385
w8385 approved these changes May 4, 2024
View reviewed changes
Copy link
Member

@w8385 w8385 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

L
G
T
M

@ssoxong ssoxong merged commit 4a18d65 into dev May 4, 2024
5 checks passed
@ssoxong ssoxong deleted the feat/transaction branch May 4, 2024 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@w8385 w8385 w8385 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

거래내역 파일 비밀번호
2 participants
@ssoxong @w8385