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

[BLOOM-138] refactor: 알림 기능 FCM -> Expo API 변경 #138

Merged
merged 3 commits into from
Oct 24, 2024
Merged

[BLOOM-138] refactor: 알림 기능 FCM -> Expo API 변경 #138

merged 3 commits into from
Oct 24, 2024

Conversation

stophwan
Copy link
Member

Close #137

How

  • 프론트엔드 요구사항에 따라 기존 FCM으로 환경을 전송하던 배치 환경을 Expo API로 변경합니다. Expo API로 변경함에 따라 프론트엔드는 더 쉽게 안드로이드, 애플 알림을 처리할 수 있습니다.
  • 기존 ErrorType에는 정해진 enum 타입과 메시지로 인해 정해진 에러 메시지를 출력하는 문제가 있어, 비즈니스 로직에서 커스텀에러 메시지를 보낼 수 있게 ExternalException을 변경했습니다.
  • Expo API는 한번의 요청에 json 형태로 100명의 사용자를 묶어서 요청을 보낼 수 있습니다. 따라서 chunk를 100으로 설정하고 writer에서 100명씩 보낼 수 있도록 하였습니다.
  • 알림을 비동기로 처리하고 바로 다음 chunk로 이동하는 방식은 기존과 동일합니다.

@stophwan stophwan added the 💊 refactor Suggest code improvements or refactoring label Oct 24, 2024
@stophwan stophwan requested a review from Dompoo as a code owner October 24, 2024 02:08
@stophwan stophwan self-assigned this Oct 24, 2024
@stophwan stophwan changed the title Refactor/expo [BLOOM-138] refactor: 알림 기능 FCM -> Expo API 변경 Oct 24, 2024
@stophwan stophwan merged commit 5bdaa9c into main Oct 24, 2024
1 check passed
@stophwan stophwan deleted the refactor/expo branch October 24, 2024 02:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Dompoo Dompoo Awaiting requested review from Dompoo Dompoo is a code owner

Assignees

@stophwan stophwan

Labels
💊 refactor Suggest code improvements or refactoring
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[refactor] 알림 기능 FCM -> Expo API 변경
1 participant
@stophwan