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

Firebase Cloud Messaging 탑재 #12

Merged
merged 3 commits into from
Jan 8, 2025
Merged

Firebase Cloud Messaging 탑재 #12

merged 3 commits into from
Jan 8, 2025

Conversation

Eunice0927
Copy link
Owner

@Eunice0927 Eunice0927 commented Jan 4, 2025

개요

변경 사항

  • 1.0.3 이상 버전을 사용하는 유저에게 푸시 메세지를 발송할 수 있습니다.
  • 메세지 클릭시 앱이 실행됩니다.

공유사항(배운 것, 참고하면 좋을 것)

스크린샷

제목 실행 중 종료시
푸시메세지 수신 화면

전달사항

@Eunice0927 Eunice0927 added the ✨ Feat 새로운 기능을 구현했어요 label Jan 4, 2025
@Eunice0927 Eunice0927 requested a review from qlrmr111 January 4, 2025 08:30
@Eunice0927 Eunice0927 self-assigned this Jan 4, 2025
@trafico-bot trafico-bot bot added the 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 label Jan 4, 2025
@Eunice0927 Eunice0927 linked an issue Jan 4, 2025 that may be closed by this pull request
@trafico-bot trafico-bot bot added the ⭐️ 운기 Pull Request Reviews assigned to GitHub User: qlrmr111 label Jan 4, 2025
Copy link
Collaborator

@qlrmr111 qlrmr111 left a comment

Choose a reason for hiding this comment

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

크게 손볼게 없네요 완벽합니다 혹시 궁금한게 그러면 알림 권한은 어떻게 하나요?
지금은 느린우체통 편지 저장할때 알림 권한 확인하는데, 이젠 앱 처음 실핼할 때 부터 받아와야 하죠?

@@ -0,0 +1,33 @@
//
// AppDelegate+MessagingDelegate.swift
Copy link
Collaborator

Choose a reason for hiding this comment

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

파일 이름이랑 여기 이름이 다르네요 하나로 통일 하는게 괜찮을거 같아요 (오 근데 역할이 두개인걸 한꺼번에 넣으면 +로도 이름 정하나보네요)

Copy link
Owner Author

Choose a reason for hiding this comment

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

역할이 두개라기보다는 어떤것을 분리할때 사용하는 것 같아요.
주로 extension을 파일 분리할 때 저는 많이 본 것 같습니다!

@trafico-bot trafico-bot bot added 👍 Merge 진행 PR 승인이 완료되어 merge할 수 있어요 and removed 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 ⭐️ 운기 Pull Request Reviews assigned to GitHub User: qlrmr111 labels Jan 8, 2025
@Eunice0927
Copy link
Owner Author

크게 손볼게 없네요 완벽합니다 혹시 궁금한게 그러면 알림 권한은 어떻게 하나요? 지금은 느린우체통 편지 저장할때 알림 권한 확인하는데, 이젠 앱 처음 실핼할 때 부터 받아와야 하죠?

오 이부분 고려하지 못했었네요. 코드 수정 필요할 것 같습니다!

@trafico-bot trafico-bot bot added 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 and removed 👍 Merge 진행 PR 승인이 완료되어 merge할 수 있어요 labels Jan 8, 2025
@Eunice0927 Eunice0927 merged commit 7b7e00f into dev Jan 8, 2025
@Eunice0927 Eunice0927 deleted the feat/AddFCM branch January 8, 2025 11:20
@trafico-bot trafico-bot bot added 👏 Merge 완료 PR이 성공적으로 merge되었어요 and removed 👀 리뷰 요청 PR이 아직 리뷰되지 않고 있어요 labels Jan 8, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feat 새로운 기능을 구현했어요 👏 Merge 완료 PR이 성공적으로 merge되었어요
Projects
None yet
Development

Successfully merging this pull request may close these issues.

FCM 탑재
2 participants