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.

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

Feat/transaction #20

Merged
merged 7 commits into from
Apr 27, 2024
Merged

Feat/transaction #20

merged 7 commits into from
Apr 27, 2024

Conversation

ssoxong
Copy link
Contributor

@ssoxong ssoxong commented Apr 26, 2024

업데이트 유형

  • Hot Fix
  • Release
  • Develop
  • Others

업데이트 개요

업데이트 요약

  • transaction module 추가
  • group controller/service
    • 거래내역 엑셀파일 업로드
    • groupId별 거래내역 조회 (모두/시간범위별)
  • transactions collection 시계열 데이터 형식으로 생성
    db.createCollection("transactions",{
      timeseries:{
        timeField: "timestamp",
        metaField: "metadata",
        granularity: "hours"
      }
    })
    
    datagrip에서 transactions과 system.buckets.transactions collecion(버킷별) 생성됨
    image
    hours 기준으로 bucket 생성되므로 행 수가 다름!! (몇시간 단위인지는 mongodb 최적화에 따라 다름)
  • groupId: 6628e63a2c217fe65efae8f9로 테스트데이터 넣어놨어용

해결한 문제

  • group controller 멤버 명단 업로드 API에 파일 업로드가 안되길래 @apiFile('memberFile') 추가

미해결 문제

  • 엑셀 파싱 방법 고려.. (현재는 카카오뱅크 기준으로 하드하게 작성)
  • event의 startDate/endDate와 입금기한은 다르므로 거래내역 조회를 위한 입금 기한을 어떻게 입력받을지/저장할지 생각을..

테스트

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

수정 사항 진단

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

@ssoxong ssoxong self-assigned this Apr 26, 2024
@ssoxong ssoxong requested a review from w8385 April 26, 2024 07:40
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.

수고했으~
버킷 생성되는거 대해서 월요일에 자세하게 설명해줘!

@ssoxong ssoxong merged commit e58bda0 into dev Apr 27, 2024
5 checks passed
@ssoxong ssoxong deleted the feat/transaction branch April 27, 2024 06:46
ssoxong added a commit that referenced this pull request Apr 29, 2024
* feat: transaction module

* feat: find transactions by period

* fix: npm test

* fix: transaction.interface.ts

* refactor: change function name

* refactor: change dto and func name

* fix: memberExcel > memberFile

---------

Co-authored-by: 박근형 <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

모임 거래내역 엑셀 파일 올리기
2 participants