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: FCM 설정 #29

Merged
merged 4 commits into from
Feb 7, 2025
Merged

feat: FCM 설정 #29

merged 4 commits into from
Feb 7, 2025

Conversation

devmizz
Copy link
Contributor

@devmizz devmizz commented Feb 2, 2025

📌 Related Issue

🚨 해결하려는 문제가 무엇인가요?

  • FCM 연동

⭐️ 어떻게 해결했나요?

FCM 연동

  • FCM 연동 작업만 진행했습니다.
  • FCM 메세지 발송하는 도메인에서 infrastructure에서 DI를 통해 적절히 구현하는 작업이 필요해보입니다.
  • 당장에는 알람 발송하는 부분이 없어 따로 구현하지 않았습니다.

FCM 토큰 저장

  • FCM 연동을 진행하면서 토큰을 저장할 테이블이 없어서 별도로 생성하였습니다.
  • User에 두지 않고 UserDevice 테이블을 따로 생성 및 저장한 이유는 다음과 같습니다.
    • 추후 여러 기기에 알람을 보낼 수 있으려면, 유저의 기기 정보를 담아둘 테이블이 따로 필요하다.
    • FCM 토큰은 기기에 종속적이므로 함께 보관하는 것이 좋다.

FcmClientInTest

  • CI 시에 test profile로 실행이 됩니다.
  • test yaml에 FCM 연동 관련 정보를 노출할 수 없어서, CI 시에는 실제 연동이 필요하지 않도록 Stub 객체를 생성했습니다.
  • FcmConfig도 test 환경에서는 Bean으로 등록되지 않습니다.

🤔 어떤 부분에 집중하여 리뷰해야 할까요?

🗒️ 참고자료

RCA 룰

  • R: 꼭 반영해 주세요. 적극적으로 고려해 주세요. (Request changes)
  • C: 웬만하면 반영해 주세요. (Comment)
  • A: 반영해도 좋고 넘어가도 좋습니다. 그냥 사소한 의견입니다. (Approve)

@devmizz devmizz added the 기능 New feature or request label Feb 2, 2025
@devmizz devmizz self-assigned this Feb 2, 2025
Copy link

@Park-Young-Hun Park-Young-Hun left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!

@devmizz devmizz merged commit 50444ad into develop Feb 7, 2025
1 check passed
@devmizz devmizz deleted the feature/fcm branch February 7, 2025 07:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
기능 New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants