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

Batch: ✨ 매월 목표 금액 설정 공지 푸시 알림 배치 #141

Merged
merged 9 commits into from
Jul 25, 2024

Conversation

psychology50
Copy link
Member

작업 이유

  • 매월 1일이 되면, 지출 관리 푸시 알림을 허용한 사용자의 활성화된 device_token 전체에 푸시 알림을 보낼 수 있어야 한다.
  • 사용자가 여러 device_token을 가지고 있더라도, 사용자에게 저장되는 푸시 알림은 하나여야 한다.

작업 사항

  • 다른 건 이전 배치 작업이랑 똑같은데, 한 가지...문제가...이 공지 알림은 제목에 날짜가 들어간당...ㅎㅎ
    • 생성할 땐, 시스템 상의 현재 날짜의 달을 삽입
    • 조회할 땐, 데이터의 created_at 정보의 달을 삽입
  • 파싱할 때, announcement enum class를 호출하는 측에서 분기 처리해야 함
    • AnnounceNotificationDto의 정적 팩토리 메서드 내에서 title 만들 때, 번거롭게 또 분기처리....

image

image

저장 & 조회 모두 잘 되는 것을 확인 함.


리뷰어가 중점적으로 확인해야 하는 부분

  • MONTHLY_TARGET_AMOUNT 상수의 제목 파싱하는 방법이 너무 더럽고, 휴먼 에러를 야기할 수 있다고 생각.
    • 하지만 반면으로 "공지" 알림이라는 게 빈번한 수정이 발생할 사안은 아닌 듯 해서 냅둬도 괜찮을 지도.
    • 분기처리를 Announcement 호출자 측으로 위임해버리는 바람에, title, content 수정 시 버그가 많아질 것 같아서 대비책을 구상해둘 필요가 있을 것 같음..

발견한 이슈

  • 없음.

@psychology50 psychology50 added the enhancement New feature or request label Jul 25, 2024
@psychology50 psychology50 self-assigned this Jul 25, 2024
@psychology50 psychology50 merged commit 9175dfa into dev Jul 25, 2024
2 checks passed
@psychology50 psychology50 deleted the feat/PW-424-target-amount-push-notify-batch branch July 25, 2024 16:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant