[Feature/#68] : fcm, like api #72

sohyun127 · 2024-09-05T08:53:20Z

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

merge할 브랜치의 위치를 확인해 주세요(main❌/develop⭕)
리뷰가 필요한 경우 리뷰어를 지정해 주세요
P1 단계의 리뷰는 필수로 반영합니다.
Approve된 PR은 assigner가 머지하고, 수정 요청이 온 경우 수정 후 다시 push를 합니다.

📌 𝗜𝘀𝘀𝘂𝗲𝘀

closed [FEAT] : FCM, like api #68

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

피드, 답글 좋아요 api
fcm
앱 로고 넣기
답글 작성 시 아래로 스크롤

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

좋아요

20240905_174735.mp4

답글 작성 완료시

scroll.webm

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

일단 요기까지 해서 QA 받을게요!
좋아요 기능은 여러번 클릭시 API 반복 호출을 막기 위해 디바운스를 사용했습니다. 이벤트 종료 시점 0.5 동안 추가 이벤트 없으면 api 호출하는 형태입니다!
fcm은 좀 수정해야할 거 같아요!

…into feature/#68-fcm-like-api

…BLE-ANDROID into feature/#68-fcm-like-api

…into feature/#68-fcm-like-api

Eonji-sw

Eonji-sw · 2024-09-06T01:09:47Z

core/network/src/main/java/com/teamwable/network/datasource/ProfileService.kt

@@ -29,6 +29,7 @@ interface ProfileService {
    @PATCH("api/v1/withdrawal")
    suspend fun patchWithdrawal(@Body requestWithdrawalDto: RequestWithdrawalDto): BaseUnitResponse<Unit>

+    @Multipart


P3 : 아! 헷

Eonji-sw · 2024-09-06T01:13:26Z

feature/home/src/main/java/com/teamwable/home/HomeFragment.kt

@@ -159,4 +174,44 @@ class HomeFragment : BindingFragment<FragmentHomeBinding>(FragmentHomeBinding::i
            if (isUploaded) scrollToTop()
        }
    }
+
+    private val requestPermission = registerForActivityResult(


P3 : 다 합치고 알림도 갤러리와 같이 첨 시작할 때 받을건가요?

갤러리 동의는 온보딩 중 프로필 설정 뷰에서 받고, 푸시 알림 동의는 홈에서 받기로 했습니다! (기획 요청)

Eonji-sw · 2024-09-06T01:16:35Z

feature/home/src/main/java/com/teamwable/homedetail/HomeDetailFragment.kt

-            toast("like")
+        override fun onLikeBtnClick(viewHolder: FeedViewHolder, id: Long, isLiked: Boolean) {
+            feedActionHandler.onLikeBtnClick(viewHolder, id) { feedId, likeState ->
+                singleEventHandler.debounce(coroutineScope = lifecycleScope) {


P3 : debounce 말고 throttle는 어때요?

debounce와 throttle의 차이점은 throttle은 적어도 n 밀리 초마다 정기적으로 기능 실행을 보장한다는 점입니다!
디바운스는 아무리 많은 이벤트가 발생해도 모두 무시하고 특정 시간사이에 어떤 이벤트도 발생하지 않았을 때 딱 한번만 마지막 이벤트를 발생시키는 기법이구요!

throttle을 쓴다면 좋아요를 연속해서 누르는 행위를 완전히 막기 보다는 api 호출을 줄여준다는 표현이 맞을 것 같네요! throttle은 일정 시간마다 요청을 보내므로 여러번 클릭 시 그 간격에 따라 요청이 발생할 수 도 있습니다 그래서 debounce로 구현했습니당

Eonji-sw

까먹음 ㅋㅋ

chanubc

천재 개발자의 개미친코드 잘보고 갑니다 🥇

chanubc · 2024-09-06T00:56:47Z

app/build.gradle.kts

+    implementation(libs.google.play.services)
+    implementation(libs.firebase.messaging.ktx)


혹시 이거 최신 버전에서 돌아가나요? z플립6에서 fcm안오길래 버전 업하니까 오긴 했습니다
저도 정확한 원인은 모르겠어요ㅠ

아직 푸시알림 안될거에요 ㅠ.ㅠ 수정 중 ㅎㅎ 다음 PR에서 수정하겠습니다-!!

chanubc · 2024-09-06T00:58:18Z

app/src/main/AndroidManifest.xml

+        <meta-data
+            android:name="com.google.firebase.messaging.default_notification_color"
+            android:resource="@color/white" />


이번엔 meta data에 icon추가 안 해도 되나요?

추가해야합니다!!! 수정하다가 빼놓고 추가 안했네요 ..ㅎㅎ

chanubc · 2024-09-06T00:58:50Z

app/src/main/java/com/teamwable/wable/service/WableFirebaseMessagingService.kt

그대로 복붙했는데 돌아가는 미친코드 👍 👍 👍

chanubc · 2024-09-06T01:03:11Z

core/ui/src/main/java/com/teamwable/ui/util/CommentActionHandler.kt

+        }
+
+        viewHolder.likeCountTv.text = updatedLikeCount.toString()
+        saveLike(id, LikeState(viewHolder.likeBtn.isChecked, updatedLikeCount.toString()))


chanubc · 2024-09-06T01:38:37Z

feature/profile/src/main/java/com/teamwable/profile/profiletabs/ProfileFeedListViewModel.kt

+                likeFeedsFlow.value = likeFeedsFlow.value.toMutableMap().apply {
+                    put(feedId, likeState)
+                }


Suggested change

likeFeedsFlow.value = likeFeedsFlow.value.toMutableMap().apply {

put(feedId, likeState)

}

likeFeedsFlow.update { currentMap ->

currentMap.toMutableMap().apply {

put(feedId, likeState)

}

}

만약 likeFeedsFlow 여러군데서 접근이 된다면 update 함수를 통해 스레드 안정성을 줄 수 있다고 합니다!
사실 큰 차이는 모르겠는데 저도 값 변화 많은 곳들은 이번에 upate사용해 보앗어요

오!!!! 정말 좋은 지식이네요!! 배우고 갑니다-! 멋지네요

반영했습니다! 25bcd3a

chanubc · 2024-09-06T01:39:15Z

feature/home/src/main/java/com/teamwable/homedetail/HomeDetailViewModel.kt

+                .filter { removedCommentIds.contains(it.commentId).not() }
+                .map { data ->
+                    val likeState = likeStates[data.commentId] ?: LikeState(data.isLiked, data.likedNumber)
+                    val transformedGhost = if (ghostedUserIds.contains(data.postAuthorId)) data.copy(isPostAuthorGhost = true) else data
+                    transformedGhost.copy(likedNumber = likeState.count, isLiked = likeState.isLiked)
+                }


나중에 요긴 usecase로 뺄수도 있을것 같아요!

넘 좋아요! 옛날에 usecase 쓸 때는 필요성을 잘 몰랐는데, 요즘 wable하면서 필요성을 너무 느끼고 있습니다 ㅋㅋㅋㅋㅋ
다음 스프린트 때 도입해봐요 ~!

sohyun127 · 2024-09-06T09:29:05Z

FCM은 다음 pr에서 완료하겠습니다~!

sohyun127 added 20 commits September 5, 2024 04:09

#68 [FEAT] : set up feed like api

0f7f218

#68 [FEAT] : set up like single event

0a73bad

#68 [FEAT] : set up like logic

4a8dd3d

#68 [MOD] : change function name

158611f

#68 [FEAT] : set up comment like api

68a3136

#68 [MOD] : change name

e72473e

#68 [FEAT] : set up comment like action handler

295fe73

#68 [FEAT] : set up comment like logic at profile comment

63b327b

#68 [MOD] : change name

7e49059

#68 [FEAT] : set up home detail recycler view divider

9f1aaa9

#68 [FEAT] : set up home detail like logic, comment scroll to bottom

c70bd98

#68 [FIX] : fix home detail divider

0f06c60

Merge branch 'develop' of https://github.com/Team-Wable/WABLE-ANDROID …

38248e8

…into feature/#68-fcm-like-api

#68 [FEAT] : set up fcm alert dialog

7d2920b

#68 [FEAT] : set up app logo

b8f648f

Merge branch 'feature/#66-splash' of https://github.com/Team-Wable/WA…

a832ea8

…BLE-ANDROID into feature/#68-fcm-like-api

#68 [HOTFIX] : fix multipart anotation

987e20f

#68 [FEAT] : set up fcm logic

9d34ae1

#68 [FIX] : fix block double click comment post

4f232d1

#68 [FIX] : fix fcm logic

f762007

sohyun127 added ⭐ [FEAT] 새로운 기능 구현 🐱 소현 리나 labels Sep 5, 2024

sohyun127 requested review from a team, Eonji-sw and chanubc and removed request for a team September 5, 2024 08:53

auto-assign bot assigned sohyun127 Sep 5, 2024

sohyun127 added 2 commits September 5, 2024 17:55

Merge branch 'develop' of https://github.com/Team-Wable/WABLE-ANDROID …

20739a9

…into feature/#68-fcm-like-api

#68 [FIX] : fix ci error

73691b4

Eonji-sw reviewed Sep 6, 2024

View reviewed changes

Eonji-sw approved these changes Sep 6, 2024

View reviewed changes

chanubc approved these changes Sep 6, 2024

View reviewed changes

sohyun127 added 2 commits September 6, 2024 18:14

#68 [HOTFIX] : fix home state error

f82fa72

#68 [REFACTOR] : change likefeeds with update function

25bcd3a

sohyun127 merged commit 4afbb68 into develop Sep 6, 2024
1 check passed

sohyun127 deleted the feature/#68-fcm-like-api branch September 7, 2024 15:35

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[Feature/#68] : fcm, like api #72

[Feature/#68] : fcm, like api #72

sohyun127 commented Sep 5, 2024

Eonji-sw left a comment

Eonji-sw Sep 6, 2024

Eonji-sw Sep 6, 2024

sohyun127 Sep 6, 2024

Eonji-sw Sep 6, 2024

sohyun127 Sep 6, 2024

Eonji-sw left a comment

chanubc left a comment

chanubc Sep 6, 2024

sohyun127 Sep 6, 2024

chanubc Sep 6, 2024

sohyun127 Sep 6, 2024

chanubc Sep 6, 2024

chanubc Sep 6, 2024

chanubc Sep 6, 2024

sohyun127 Sep 6, 2024

sohyun127 Sep 6, 2024

chanubc Sep 6, 2024

sohyun127 Sep 6, 2024

sohyun127 commented Sep 6, 2024

		implementation(libs.google.play.services)
		implementation(libs.firebase.messaging.ktx)

[Feature/#68] : fcm, like api #72

[Feature/#68] : fcm, like api #72

Conversation

sohyun127 commented Sep 5, 2024

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

📌 𝗜𝘀𝘀𝘂𝗲𝘀

📎𝗪𝗼𝗿𝗸 𝗗𝗲𝘀𝗰𝗿𝗶𝗽𝘁𝗶𝗼𝗻

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

좋아요

답글 작성 완료시

💬 𝗧𝗼 𝗥𝗲𝘃𝗶𝗲𝘄𝗲𝗿𝘀

Eonji-sw left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Eonji-sw left a comment

Choose a reason for hiding this comment

chanubc left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sohyun127 commented Sep 6, 2024