[FEAT/#382] FCM 푸시알림 커스텀 / Payload 형식에 맞게 구현 #383

leeeyubin · 2025-04-28T17:06:36Z

closed [FEAT] FCM 푸시알림 커스텀 / Payload 형식에 맞게 구현 #382

⛳️ Work Description

Payload 형식 수정
푸시알림에 LargeIcon 추가
알림 유형별 분기처리
포그라운드 상태일 때 분기처리

📸 Screenshot

LargeIcon 적용 사진

처음 알림 클릭 시 `(탐색으로 가는 알림)`	포그라운드 상태에서 알림 클릭 시 `(홈으로 가는 알림)`
Screen_Recording_20250429_014720_One.UI.Home.mp4	Screen_Recording_20250429_014938_One.UI.Home.mp4

📢 To Reviewers

드디어 길고 길었던 푸시알림 구현이 끝났습니다!🥹
알림을 클릭했을 때 무조건 스플래쉬를 거치는 것은 맞지만, 앱이 활성화 되어있는 상태(포그라운드)일 때는 바로 그 화면으로 이동하도록 했어요!
이를 구현하면서 앱의 진입점(startDestination)을 매번 바꾸는 방향으로 수정했습니다! (아린언니 보고 참고해주세요!)
고려하지 않게 된 두 가지 사항에 대해서도 이유를 작성해보겠습니당
1. 업데이트 확인하기(포그라운드) : 알림 자체를 처음 눌렀을 때는 스플래쉬를 거치기 때문에, 굳이 사용자가 앱을 쓰고 있을 때 업데이트가 일어나는 경우는 고려할 필요가 없다고 판단.
2. 토큰 유무 확인하기 : 알림을 받기 위해서는 alarmAvailable을 설정해줘야하는데, 로그인이 안 된 상태는 무조건 alarmAvailable이 false임. 그래서 알림을 받을 케이스가 없음

boiledeggg

로직이 상당히 복잡하네요😵 4차 스프린트 동안 너무 고생많았어유~~!!

boiledeggg · 2025-04-29T08:44:03Z

core/firebase/src/main/java/com/terning/core/firebase/messageservice/TerningMessagingService.kt

+        return appProcesses?.any {
+            it.importance == ActivityManager.RunningAppProcessInfo.IMPORTANCE_FOREGROUND &&
+                    it.processName == packageName
+        } == true


조건들을 변수로 선언해두면 코드 가독성이 지금보다 좋아질 것 같아요!

반영했습니다! 가독성이 좋아진 것 같네요:)

boiledeggg · 2025-04-29T08:46:00Z

feature/search/src/main/java/com/terning/feature/search/search/SearchRoute.kt

+    val systemUiController = rememberSystemUiController()
+
+    LaunchedEffect(Unit) {
+        systemUiController.setStatusBarColor(color = White)
+        systemUiController.setNavigationBarColor(color = White)
+    }


이걸 모든 화면에 선언해준 이유는 뭔가요? 안드로이드 API 때문인가요??

이 코드는 상태바와 네비게이션바의 색상을 지정해주는 코드에요!

더 이상 앱의 진입부분이 홈 화면만이 아니게 되면서 다른 진입 화면들도 색상을 정의해줘야되더라구요
위 코드가 없으면 아래와 같은 현상이 발생했습니다..

redirect하면 splash를 거치지 않고 바로 진입하는건가요?!?? splash 통해서 이동하는 로직이면 onDispose 내부에서 white로 다시 바꿔줘도 되지 않을까 싶어서요!(테스트는 안해봐서 될지는 모르겠어잉ㅎㅎ)

헉 왜 그 생각을 못했을까요!! 잘 됩니당~! 좋은 의견 감사해요!🚀🚀

보충 설명하자면 redirect는 Splash를 거친 후 리다이렉트 할 화면을 의미합니다!

boiledeggg · 2025-04-29T08:47:56Z

feature/main/src/main/java/com/terning/feature/main/MainScreen.kt

+                            navOptions = NavOptions.Builder().setPopUpTo(
+                                route = Splash(redirect),
+                                inclusive = true
+                            ).build()


리다이렉트 관련된 화면들의 navOption들이 전부 동일한 것 같은데 하나로 통일해버리는건 어떤가요??

하나로 통일한다는 건 확장함수로 만들어서 적용하는 걸 의미하는 걸까요..?!

각각 다른 객체를 만들어 넘겨줄 필요 없이 변수 하나 만들어두고 사용하면 되지 않을까 싶어요!

이해했습니다!! 반영했어요! 역시 가독성의 신 석준오빠..🚀🚀

Hyobeen-Park

으아 진짜 어렵네요.. 우리 리드 짱이다 멋있다 최고다🚀🚀

Hyobeen-Park · 2025-04-30T05:18:58Z

feature/search/src/main/java/com/terning/feature/search/search/SearchRoute.kt

+    val systemUiController = rememberSystemUiController()
+
+    LaunchedEffect(Unit) {
+        systemUiController.setStatusBarColor(color = White)
+        systemUiController.setNavigationBarColor(color = White)
+    }


redirect하면 splash를 거치지 않고 바로 진입하는건가요?!?? splash 통해서 이동하는 로직이면 onDispose 내부에서 white로 다시 바꿔줘도 되지 않을까 싶어서요!(테스트는 안해봐서 될지는 모르겠어잉ㅎㅎ)

leeeyubin added 10 commits April 28, 2025 00:17

[FEAT/#382] imageUrl 추가

b16f0b8

[FEAT/#382] 푸시알림 LargeIcon 구현

6214e64

[FEAT/#382] 상태바, 네비게이션바 색상 설정

18aacdb

[FEAT/#382] 화면이동 하는 뷰 딥링크 작성

16fed9c

[FEAT/#382] 포그라운드 여부를 포함한 알림 유형 분기처리 구현

86bc36a

[FEAT/#382] 딥링크 고정 값 작성

8dd57a1

[FEAT/#382] 리다이렉트 enum class 작성

a07c088

[FEAT/#382] :core:designsystem 의존성 추가

28b07be

[FEAT/#382] 기존 코드 효율적으로 변경

14f2a91

[FEAT/#382] MainNavigator enum class 사용

10853db

leeeyubin added FEAT ✨ 새로운 기능 구현 유빈💙 유빈 labels Apr 28, 2025

leeeyubin added this to the 4차 스프린트 작업 milestone Apr 28, 2025

leeeyubin requested review from Hyobeen-Park, arinming and boiledeggg April 28, 2025 17:06

leeeyubin self-assigned this Apr 28, 2025

leeeyubin added 2 commits April 29, 2025 02:23

[FEAT/#382] buildDeeplink 함수명 수정

a0950d9

[CHORE/#382] 한 줄 공백 제거

3d11f5f

boiledeggg approved these changes Apr 29, 2025

View reviewed changes

leeeyubin added 2 commits April 30, 2025 00:48

[CHORE/#382] 조건 변수화

565f4e0

[CHORE/#382] 줄바꿈 추가

fbf686d

Hyobeen-Park approved these changes Apr 30, 2025

View reviewed changes

leeeyubin added 2 commits April 30, 2025 16:23

[CHORE/#382] Splash 화면에 onDispose 추가

8d6da4d

[CHORE/#382] splashNavOptions 변수화

a08dc32

leeeyubin merged commit 5df3ce7 into develop Apr 30, 2025
1 check passed

[FEAT/#382] FCM 푸시알림 커스텀 / Payload 형식에 맞게 구현 #383

[FEAT/#382] FCM 푸시알림 커스텀 / Payload 형식에 맞게 구현 #383

Uh oh!

Conversation

leeeyubin commented Apr 28, 2025 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

⛳️ Work Description

📸 Screenshot

📢 To Reviewers

Uh oh!

boiledeggg left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Hyobeen-Park left a comment

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

leeeyubin commented Apr 28, 2025 •

edited

Loading