FCM 에러 해결 #469
Merged
FCM 에러 해결 #469
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
WaiNaat
approved these changes
Oct 20, 2023
| state={currentSubscribe} | ||
| disabled={!pushSupport || notificationDenied === 'denied'} | ||
| /> | ||
| <Suspense fallback={<div>로딩중잉ㅂ니다?</div>}> |
Choi-JJunho
pushed a commit
that referenced
this pull request
Oct 20, 2023
* refactor: friebase FCM 관리 모듈로 분리 * refactor: Notification 제거 및 구독 함수 수정
hozzijeong
added a commit
that referenced
this pull request
Oct 20, 2023
* refactor: friebase FCM 관리 모듈로 분리 * refactor: Notification 제거 및 구독 함수 수정
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
🔥 연관 이슈
🚀 작업 내용
💬 리뷰 중점사항
FCMMessaging이라는 클래스를 만들었습니다. 주노가 올린 에러는 firebase messaging을 지원하지 않는다는 에러인데, 이게 permission을 얻지 않은 상태에서 접근하면 저렇게 되는 것 같습니다.
ios에서 Notification이라는 API를 사용하기 위해서는 홈 화면 바로가기 에서만 사용이 가능합니다. 따라서 홈 화면 바로가기가 아닌 경우에는 분기처리가 필수적으로 필요합니다. 이에 Notification을 사용하는 코드에 맞춰 분기처리를 진행했습니다.
근데 이상한 것은 window에 notification이라는 객체가 없으면 serviceWorker를 등록하지 못하게 설정했음에도 이게 무시되고 에러를 터뜨린다는 것이좀 의아합니다. 또한 serviceWorker를 등록할 때 Notification을 통해 permission을 얻지 않아도 딱히 에러가 발생하지 않는다는게 참 신기하면서도 불안한 부분입니다.