Skip to content

Conversation

@ChaeYubin
Copy link
Contributor

📑 작업 개요

  • 거북목 주의보 알림 관련 버그 수정

✨ 작업 이유

  • 기존 로직은 회원가입 이후에 FCM 토큰을 발급받아 서버에 등록했으나, 회원가입 시에 알림을 허용해두지 않은 상태인 사용자가 있을 수 있고, 따라서 FCM 토큰이 제대로 저장되지 않을 수 있었습니다.
  • FCM 토큰 값이 일관적이지 않은 주기로 재발급되기 때문에, 이를 갱신해주지 않을 시 푸시 알림을 제대로 받을 수 없는 문제가 있었습니다.

📌 작업 내용

  • FCM 토큰 발급 함수 유틸 함수로 추출
  • 거북목 주의보 알림 버튼 토글 시 서버에 저장된 FCM 토큰 값과 비교하여 토큰 값 갱신하는 로직 추가

🤝🏻 해당 부분을 중점적으로 리뷰해주세요!

궁금한 점있으시면 코멘트 남겨주세요!
+) 현재 알림 주기가 1시간이 아닌 1분으로 설정되어 있습니다. dev 배포 버전에서 확인 후 1시간으로 다시 수정 요청 드릴 예정입니다.

🖥️ 실행 화면

-.Clipchamp.11.mp4

🎟️ 관련 이슈

close: #312

@ChaeYubin ChaeYubin self-assigned this Mar 17, 2025
@ChaeYubin ChaeYubin requested review from hvrain and typeYu March 17, 2025 06:39
Copy link
Contributor

@hvrain hvrain left a comment

Choose a reason for hiding this comment

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

회원가입할 때 알림 요청할 경우 사용자가 알림을 허용할 이유를 몰랐는데, 그 부분도 같이 해결된 것 같네요. 구현 수고하셨습니다.

Copy link
Contributor

@typeYu typeYu left a comment

Choose a reason for hiding this comment

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

이제 알람을 켤 때 FCM 토큰 값을 갱신해주니, 기존 계정에서도 알람 설정이 토글 되고, 크롬 자체에서 알람을 비활성화 했더니 알람을 켜주라고 토스트 메시지가 잘 뜨네요.
그런데, 거북목 알림 자체가 1분마다 오지 않는 것 같습니다. 혹시 1시간으로 다시 변경하신건가요?

@ChaeYubin
Copy link
Contributor Author

이제 알람을 켤 때 FCM 토큰 값을 갱신해주니, 기존 계정에서도 알람 설정이 토글 되고, 크롬 자체에서 알람을 비활성화 했더니 알람을 켜주라고 토스트 메시지가 잘 뜨네요. 그런데, 거북목 알림 자체가 1분마다 오지 않는 것 같습니다. 혹시 1시간으로 다시 변경하신건가요?

엇 아뇨 아직 말씀 안드렸는데 제가 다시 한번 확인해보겠습니다

@ChaeYubin
Copy link
Contributor Author

@typeYu 하임님 혹시 푸시 알림 수신이 아예 안 되셨던 걸까요? 저는 다시 실행해보니 1분마다 알림이 오고 있어서요!

@ChaeYubin ChaeYubin merged commit 2acccf9 into dev Mar 18, 2025
1 check passed
@ChaeYubin ChaeYubin deleted the DEV-173 branch March 18, 2025 02:32
@typeYu
Copy link
Contributor

typeYu commented Mar 18, 2025

@typeYu 하임님 혹시 푸시 알림 수신이 아예 안 되셨던 걸까요? 저는 다시 실행해보니 1분마다 알림이 오고 있어서요!

2025.03.18 논의 후 배포 버전에서도 확인해 보기로 결정

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.

[DEV-173] 거북목 주의보 알림 관련 API 수정 반영

4 participants