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

[MS-191] Fcm APNs Setting #97

Merged
merged 4 commits into from
Jul 30, 2024
Merged

[MS-191] Fcm APNs Setting #97

merged 4 commits into from
Jul 30, 2024

Conversation

tjdgns8439
Copy link
Member

@tjdgns8439 tjdgns8439 commented Jul 28, 2024

요약 🎀

  • Fcm APNs 로 Sound와 content-available 설정을 해주었습니다.

상세 내용 🌈

스크린샷 2024-07-28 오후 4 25 04 스크린샷 2024-07-28 오후 3 41 37
  • 백엔드에서 FCM 서버로 메시지를 보내면 FCM 서버는 각각 요청에 맞춰 APNs 혹은 Android와 같이 플랫폼 레벨의 Layer를 추가해 메시지를 전송합니다.
  • 여기서 백그라운드 설정이 따로 되어있지 않아 프론트에서 백그라운드 메시지를 catch하여 작업함에 어려움이 있었습니다.
  • 이에 따라 아래 두 정보를 취합하여 Sound(백그라운드 진동) 설정과 함께 필드를 추가하였습니다.
  • package com.google.firebase.messaging;
  • iOS Background Notification troubleshooting invertase/react-native-firebase#3367 (comment)

질문 및 이외 사항 🚩

  • APNs설정은 iOS 설정입니다. 이에 따라 안드에 미칠 후폭풍은 고려하지 않은 상태이며 만약 안드의 설정이 저번 Notification설정 때와 같이 충돌이 난다면 아무래도 interface화 하여 아예 코드를 분리하는 것도 좋을 것 같습니다.

리뷰어에게 ✨

@tjdgns8439 tjdgns8439 added the enhancement New feature or request label Jul 28, 2024
@tjdgns8439 tjdgns8439 self-assigned this Jul 28, 2024
Copy link
Contributor

@YooJisu826 YooJisu826 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
Member

@jaewonLeeKOR jaewonLeeKOR left a comment

Choose a reason for hiding this comment

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

보라 말대로 함수로 빼도 좋을거 같네요.👍

@tjdgns8439 tjdgns8439 merged commit d975dd5 into develop Jul 30, 2024
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants