Refactor/#461 alarm

[rhkrwngud445]

🔥 연관 이슈

• close: 알림기능 자잘한 버그 수정 및 리팩토링을 한다 #461

📝 작업 요약

알람 포그라운드 버그 수정 및 함수 분리

🔎 작업 상세 설명

🌟 리뷰 요구 사항

5분이면 충분합니다

[github-actions]

Test Results

43 tests  ±0   43 ✔️ ±0   0s ⏱️ ±0s
  8 suites ±0     0 💤 ±0 
  8 files   ±0     0 ❌ ±0 

Results for commit 6271e55. ± Comparison against base commit f9823ac.

♻️ This comment has been updated with latest results.

[sujin9]

안녕하세요 코비! :)
함수 분리가 되니 보기 더 편해져서 좋네요 :)

다만,
앱을 실행 중인 상태에서, 게시글 A와 B 순서대로 추천 반응을 남기면 알람 두 개가 와요
이 알람 두 개를 눌러보면 둘 다 게시글 B로 이동합니다.
코비도 이 부분 한 번 테스트해보시고 확인해주시면 좋겠네요 :)
(저는 API 33 버전에서 테스트 진행했습니다)

+) 슬랙 버그제보에 남겨드린 체크리스트를 한 번씩 확인해보면서 버그 체크해보면 좋겠네요!
이 부분은 만나서 같이 해도 좋을 것 같습니다 ;)

[sujin9]

onNewToken을 override 해준 이유는 무엇인가요?

[sujin9]

안 쓰는 로그는 삭제해주세요 :)

[sujin9]

Suggested change

	0,
	System.currentTimeMillis().toInt(),

requestCode 가 동일한 경우에는 마지막 PendingIntent 로 치환됩니다.
그래서 제가 이해한 바로는, pendingIntent에 들어가는 requestCode는 다른 PendingIntent와 구별하기 위해 사용되는,
마치 고유값과 같은 개념인 것 같아요.

그래서 requestCode에 다른 값이 들어가도록 하여 구분을 지어보니,
리뷰에 남겨드렸던 문제가 해결되는 것 같아요.
(코비의 환경에서도 잘 동작하는지 테스트 한 번 해주세요!)

requestCode를 저는 다음과 같이 코드 수정을 제안드렸는데,
주 요점은 requestCode가 다르게 하여 pendingIntent가 구분되어 오도록 하는 것이니 더 나은 방법이 있다면 그 방법대로 해주셔도 됩니다!

ref.
https://stackoverflow.com/questions/21526319/whats-requestcode-used-for-on-pendingintent
https://devmingsa.tistory.com/11
https://iw90.tistory.com/221

[sujin9]

수정된거 확인했습니다. notificationId를 쓰니 더 직관적이고 좋은 것 같아요!! 굿굿 어푸루브 할게요 ~~
안쓰는 로그들은 제가 삭제햇서욥 !