Feature/#121 news number api #124

Eonji-sw · 2024-11-25T17:31:06Z

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

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

📌 𝗜𝘀𝘀𝘂𝗲𝘀

closed [FEAT] : news 페이지 뉴스, 공지사항 개수 API 구현 #121

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

news 페이지 뉴스, 공지사항 개수 api 구현
news 페이지 공지사항 max line 및 SpaceBetween 적용

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

2024-11-26.2.28.47.mov

2024-11-26.2.42.14.mov

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

진짜 오랜만에 개정색 빨고 한.. flow 개싫어요

server number가 local number보다 클 때에만 뱃지 띄우는거 맞죠?
로그는 머지 직전에 지울게요 진짜 쟤 안찍었으면 pr 못올렸음

전적으로 도움 준 찬우햄께 영광을 ^^..
소식 페이지 공지사항 title, content 원래 18, 50자 제한인데 망할 compose는 그게 쉽지 않아서 일단 maxline으로 제약 걸었어요 쩝..

chanubc

밤늦게 고생많으셨습니다~

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

chanubc · 2024-11-25T19:45:20Z

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

+        }.launchIn(lifecycleScope)
+    }
+
+    private suspend fun saveNumberFromServerToLocal() {


요건 뷰모델의 함수로 도 갈 수 있을것 같은데
변화된 숫자 값들만 파라미터로 넘기고 나머지는 뷰모델에서 로직을 작성해도 될것 같아요

그런데 혹시 suspend로 선언한 이유가 있을까요?

저도 이 부분이 궁금합니다
그리고 서버 num 이랑 로컬 num 비교 로직을 뷰 모델에서 작성하고 그 결과만 state로 관리해도 좋을 것 같아요!

viewModel에서 flow first 땜에 해당 함수를 suspend 처리해서 이렇게 햇슴미다..
바꾸고는 싶은데 다른 플젝 땜에 오늘 안에는 수정 불가할 것 같아서 크게 문제되는게 아니라믄.. 나중에 수정하고 시퍼요

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

chanubc · 2024-11-25T19:46:54Z

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

                    }

                    NewsTabType.NOTICE.idx -> {
                        trackEvent(CLICK_NOTICE)
                        setBadgeOnNews(NewsTabType.NOTICE.idx, false)
+                        lifecycleScope.launch { viewModel.saveNoticeNumber(serverNoticeNumber) }


p3 : saveNoticeNumber가 viewmodel scope에서 실행되는 함수라면 코루틴 스코프 중첩은 불필요해보여요!

그르네요 굿!

chanubc · 2024-11-25T19:49:38Z

feature/news/src/main/java/com/teamwable/news/NewsViewModel.kt

+    suspend fun saveNewsNumber(newsNumber: Int) = userInfoRepository.saveNewsNumber(newsNumber)
+


save는 suspend 보단 viewmodel scope내에서 처리하는게 어떠신지?
suspend는 스레드 블락의 개념이 아니라 컨택스트 스위칭 및 cpu에서 제어권을 기억하는 리소스 낭비는 없지만

저장의 경우 중단,재개로 작업을 이어나갈 필요는 없을것 같다 생각해요!

그리고 suspend함수의 호출 부는 코루틴 빌더 내부에서 실행을 시켜야 하기 때문에 결국 비슷한 작업이라 생각합니다!

이렇게 되면 불안정한 생명주기를 가진 view단 보단 viewmodelscope에서 io 작업을 처리하는게 더 나아보여요!

아하 넵!

chanubc · 2024-11-25T19:53:42Z

feature/news/src/main/java/com/teamwable/news/NewsViewModel.kt

+    suspend fun getNoticeNumberFromLocal() = userInfoRepository.getNoticeNumber().first()
+


flow는 계속해서 데이터스트림을 방출한다는 장점이 있어요!
first()보단 collect하거나 statein해서 변경값이 감지 되었으면 뱃지를 띄우는 방법도 잇을것 같아요!

저장을 해도 값이 변경되는데 datastore는 flow로 작동이 되기때문에 값이 저장이 되어도 바로 생산과 소비가 되요! 그럼 구독하는(sateflow/statein) 쪽에서 상태변화를 곧바로 알 수 있기 때문에 조금 더 효율적일 것이란 생각이 들어요

음 근데 어차피 서버에서 가져오는건 restful이니까 단방향이기 때문에 first()로 동기적으로 가져와도 상관없겠네요

feature/news/src/main/java/com/teamwable/news/notice/NewsNoticeItem.kt

chanubc · 2024-11-25T20:01:06Z

아 그리고 제목 , 몇분전 -> 이거 전 간격 10dp로 했습니다!

sohyun127

제가 자는 새벽동안 많은 일들이 있었군요

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

sohyun127 · 2024-11-26T02:07:22Z

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

+        }.launchIn(lifecycleScope)
+    }
+
+    private suspend fun saveNumberFromServerToLocal() {


저도 이 부분이 궁금합니다
그리고 서버 num 이랑 로컬 num 비교 로직을 뷰 모델에서 작성하고 그 결과만 state로 관리해도 좋을 것 같아요!

feature/news/src/main/java/com/teamwable/news/NewsFragment.kt

sohyun127

남은 부분은 다음 pr에서 반영해주시면 될 것 같아요!

Eonji-sw added 15 commits November 26, 2024 01:53

#121 [FEAT] : news repository impl getNumber 구현

c3c8c72

#121 [FEAT] : userInfo repository impl news getNumber, saveNumber 구현

66e41ae

#121 [FEAT] : preference datasource impl news number 구현

859f3be

#121 [FEAT] : preference datasource news number 구현

7581d90

#121 [FEAT] : userInfo repository news getNumber, saveNumber 구현

bd49ce1

#121 [FEAT] : news number response dto 구현

85e6cd5

#121 [RENAME] : notification number dto 파일 네이밍 변경

7091458

#121 [RENAME] : notification number dto 파일 네이밍 변경

4afe4e1

#121 [FEAT] : news service getNumber 구현

ef10d3f

#121 [FEAT] : news repository getNumber 구현

d860a48

#121 [FEAT] : news get number api 연결 및 userInfo repository 연결

43c18e5

#121 [FEAT] : news get number api 연결 및 userInfo repository 연결

3ebff15

#121 [MOD] : saveNoticeNumber 매개변수명 변경

d0f8acc

#121 [MOD] : UiState success 코드 함수화

d27639f

#121 [MOD] : saveNumber 호출 위치 수정

bcbd574

Eonji-sw added ⭐ [FEAT] 새로운 기능 구현 🐰 언지 기만지 labels Nov 25, 2024

Eonji-sw self-assigned this Nov 25, 2024

#121 [UI] : news notice item title, content maxline 적용 및 spacebetween 적용

0256ff9

Eonji-sw added the ☕ [COMPOSE] 컴포즈 ui label Nov 25, 2024

chanubc approved these changes Nov 25, 2024

View reviewed changes

sohyun127 approved these changes Nov 26, 2024

View reviewed changes

Eonji-sw added 5 commits November 26, 2024 13:06

#121 [MOD] : scope 수정 및 enum class ordinal 적용

9e59687

#121 [MOD] : enum class idx 제거

c957ec8

#121 [MOD] : suspend -> scope 수정

68fd94c

#121 [MOD] : enum class ordinal 적용

552cd30

#121 [UI] : news notice item padding 추가

fae3d68

sohyun127 approved these changes Nov 26, 2024

View reviewed changes

Eonji-sw merged commit a987de3 into develop Nov 26, 2024
1 check passed

Eonji-sw deleted the feature/#121-news-number-api branch November 26, 2024 07:36

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Feature/#121 news number api #124

Feature/#121 news number api #124

Eonji-sw commented Nov 25, 2024 •

edited

Loading

chanubc left a comment

chanubc Nov 25, 2024

sohyun127 Nov 26, 2024

Eonji-sw Nov 26, 2024 •

edited

Loading

chanubc Nov 25, 2024

Eonji-sw Nov 26, 2024

chanubc Nov 25, 2024

chanubc Nov 25, 2024

Eonji-sw Nov 26, 2024

chanubc Nov 25, 2024

chanubc Nov 25, 2024

chanubc commented Nov 25, 2024

sohyun127 left a comment

sohyun127 Nov 26, 2024

sohyun127 left a comment

		suspend fun saveNewsNumber(newsNumber: Int) = userInfoRepository.saveNewsNumber(newsNumber)

		suspend fun getNoticeNumberFromLocal() = userInfoRepository.getNoticeNumber().first()

Feature/#121 news number api #124

Feature/#121 news number api #124

Conversation

Eonji-sw commented Nov 25, 2024 • edited Loading

✅ 𝗖𝗵𝗲𝗰𝗸-𝗟𝗶𝘀𝘁

📌 𝗜𝘀𝘀𝘂𝗲𝘀

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

📷 𝗦𝗰𝗿𝗲𝗲𝗻𝘀𝗵𝗼𝘁

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

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

Eonji-sw Nov 26, 2024 • edited Loading

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

chanubc commented Nov 25, 2024

sohyun127 left a comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

sohyun127 left a comment

Choose a reason for hiding this comment

Eonji-sw commented Nov 25, 2024 •

edited

Loading

Eonji-sw Nov 26, 2024 •

edited

Loading