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 TOKEN 수정 #33

Merged
merged 3 commits into from
Feb 13, 2025
Merged

feat: FCM TOKEN 수정 #33

merged 3 commits into from
Feb 13, 2025

Conversation

devmizz
Copy link
Contributor

@devmizz devmizz commented Feb 10, 2025

📌 Related Issue

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

  • 유저의 FCM 토큰 관련하여 몇 가지 조치가 필요함
    • 두 달 이상 서비스를 사용하지 않아 자동 만료 처리되는데, 이를 데이터로 반영이 필요
    • 로그아웃 시에 알람을 보내지 않도록 조치

⭐️ 어떻게 해결했나요?

  • 클라이언트에서 위에 언급된 케이들을 감지할 수 있음
  • 따라서 클라이언트가 FCM 토큰의 값을 변경할 수 있도록 PUT 메서드를 제공
  • 클라이언트에서 FCM 토큰이 직접적으로 사용되는 경우는 없으므로, 204 응답을 활용하여 별도의 body 없음

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

🗒️ 참고자료

RCA 룰

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

@devmizz devmizz added the 기능 New feature or request label Feb 10, 2025
@devmizz devmizz self-assigned this Feb 10, 2025
@devmizz devmizz merged commit 34746dd into develop Feb 13, 2025
1 check passed
@devmizz devmizz deleted the feature/update-fcm branch February 13, 2025 05:55
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.

1 participant