[feat/challenge_ui]: 챌린지 현황 뷰 ui 수정 #167

jihyun0v0 · 2024-03-15T13:14:24Z

개요

close [feat] 챌린지 현황 뷰 ui 수정 #166

작업 사항

챌린지 현황 뷰 ui 수정
편집 버튼 클릭 시 쓰레기통 아이콘 추가
쓰레기통 아이콘 추가를 위한 뷰모델 수정 (challengeViewModel에 ChallengeState내용 수정)
캘린더 배경이미지 변경, 캘린더 아이콘 디자인 변경
ChallengeFragment 함수 쪼개기 리팩토링

스크린샷(optional)

Screen_recording_20240315_221331.webm

허거덩,,, 하다보니 파일 98개나 건들였네요,,,, 매우 송구스럽습니다만 아이콘 추가가 많아서 그런 파일도 많을거에요,,,, 쓰흡,,,,

kangyuri1114

음.... 근데 파일체인지가 98개 있는게 단순히 건드린게 많아서 같지가 않아요...
이전 pr들도 그렇고 제일 처음에 있던 수정사항이 다른 pr에도 반영이 되고 있어서 제일 마지막 pr인 여기서가 가장 파일체인지가 많은 거 같은데.....
깃허브 브랜치 관련해서 뭔가 잘못되고 있는 것 같은데...... 무슨 문제인지 아실까욤..?

kangyuri1114 · 2024-03-23T11:22:46Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/ChallengeFragment.kt

-        collectChallengeState()
+        collectMainStateAndBindViews()
+        collectChallengeStateAndBindModifierButton()
+//        collectUsageGoalsAndModifierStateAndBindView()


이 함수는 왜 주석처리 되어있나요??

오모나 주석처리 제거 안된 부분이 상당하네요... 웁스 지우겠습니다~

kangyuri1114 · 2024-03-23T11:24:23Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/ChallengeFragment.kt

+    private fun initChallengeCreateButton() {
+        binding.btnChallengeCreate.setOnClickListener { }
+    }


아직 챌린지 생성하기 기능이 없어서 빈 것이라면 간단한 로그나 TODO 넣어놓아도 좋을 것 같습니당

kangyuri1114 · 2024-03-23T11:27:58Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/ChallengeViewModel.kt

+//    private val _usageGoalsState = MutableStateFlow(UsageGoalsState())
+//    val usageGoalsState = _usageGoalsState.asStateFlow()


usageGoal 관련 로직이 다 주석처리 되어있네용
지금 로그인 안되어서 오류나서 그러는건가요??

지웠습니다~

kangyuri1114 · 2024-03-23T11:28:30Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/appadd/AppAddViewModel.kt

@@ -28,6 +29,15 @@ data class AppAddState(
 class AppAddViewModel @Inject constructor(
    private val getInstalledAppUseCase: GetInstalledAppUseCase
 ) : ViewModel() {
+    private fun getInstalledApps() {
+        viewModelScope.launch {
+            delay(50)


delay는 왜 추가 되었나요??

Slack에 간단히 언급했던 문제입니당! (https://hmhteamhq.slack.com/archives/C06AHJP0R9R/p1710509172258019)

자세한 설명은 notion에 기록해두면서 해결해 나가려고 하는데(https://www.notion.so/msmmx/NullPointerException-6de7f2846c774cca9c02410a7dd5c2b1) 여기서 조금 설명을 드려보자면,
앱 진입하자마자 달성현황뷰에서 앱추가 선택 시 NullPointerException 발생하는 문제가 있었습니다.
앱을 추가하려면 앱 리스트를 시스템으로 부터 받아와야 합니다. 하지만, 이때 앱 리스트를 받아오는데 이 과정에서 리스트가 다 전달되지 않았는데 이를 사용하려 하는 경우 즉, Empty List를 참조하는 경우가 발생한다고 생각했습니다.

이를 delay를 통해 해당 Exception을 예방하고자 했습니다. (이렇게 하니 문제는 해결되었는데 궁국적인 해결책은 아닌 것 같다는 생각이 듭니다. 일단 'NullPointerException을 예방하기 위한 임시 해결책'이라고 생각해주시면 될 것 같아요!)

오 그렇군용 ! delay방법과 비슷하게 LodingView를 넣는 것도 또 다른 방법이 될 것 같아용!

kangyuri1114 · 2024-03-23T11:29:23Z

...re/challenge/src/main/java/com/hmh/hamyeonham/challenge/calendar/ChallengeCalendarAdapter.kt

 import com.hmh.hamyeonham.feature.challenge.R
 import com.hmh.hamyeonham.feature.challenge.databinding.ItemChallengeStatusBinding

-class ChallengeCalendarAdapter :
+class ChallengeCalendarAdapter(private val context: Context) :


이 context는 언제 사용되는걸까요??

해당 부분은 viewHolder에 다시 전달되어 getColor함수에서 (아이디에 따른 색상 값을 가져올 때) 사용됩니다.

kangyuri1114 · 2024-03-23T11:50:41Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/goals/ChallengeViewHolder.kt

+        fun setVisibility(modifierState: ModifierState) {
+            val editMode = if (modifierState == ModifierState.EDIT) View.INVISIBLE else View.VISIBLE
+            val doneMode = if (modifierState != ModifierState.EDIT) View.INVISIBLE else View.VISIBLE
+            binding.tvAppName.visibility = doneMode
+            binding.tvAppUsageGoal.visibility = doneMode
+
+            binding.tvAppNameDelete.visibility = editMode
+            binding.tvAppUsageGoalDelete.visibility = editMode
+            binding.ivTrash.visibility = editMode
+        }


앱 편집 상태에 따른 UI 변경 맞죠?? 확인했습니당

kangyuri1114 · 2024-03-23T11:52:43Z

첫 pr 이후 브랜치들 첫pr의 브랜치 기준으로 소브랜치 파서 작업한건가요??

…init~~ 으로 함수 명 변경

kangyuri1114 · 2024-03-24T07:32:56Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/ChallengeViewModel.kt

 enum class ModifierState {
-    DELETE,
-    CANCEL,
+    EDIT, DONE,


네이밍 변경된거 너무 좋은 것 같아용

kangyuri1114 · 2024-03-24T07:38:42Z

feature/challenge/src/main/java/com/hmh/hamyeonham/challenge/goals/ChallengeViewHolder.kt

+            binding.tvAppName.visibility = doneMode
+            binding.tvAppUsageGoal.visibility = doneMode
+
+            binding.tvAppNameDelete.visibility = editMode
+            binding.tvAppUsageGoalDelete.visibility = editMode
+            binding.ivTrash.visibility = editMode


각 텍스트들을 state에 따라 보이게, 안보이게 하는 코드 같은데
visible 보다는 텍스트뷰는 하나로 두고 각 state에 따라 string을 바꾸는게 좋지 않을까용?

이전 코드 보다 훨씬 깔끔해진 것 같아용~!

kangyuri1114 · 2024-03-28T06:55:49Z

feature/challenge/src/main/res/drawable/ic_chellenge_point_exist_24.xml

@@ -0,0 +1,94 @@
+<vector xmlns:android="http://schemas.android.com/apk/res/android"


이거 포인트 뱃지 아이콘 맞을까요????
이 아이콘 파일이 포인트 받기 뷰에서도 쓰이는데 core로 빼는 거 어떨까요??

넵 수정하겠습니당

[�feat/challenge_ui]: 챌린지 현황 뷰 ui 수정

jihyun0v0 added 16 commits March 15, 2024 15:47

[feat]: 우주상점 ui 리소스 추가

63d613f

[fix]: 버튼 이름 수정

c502437

[feat]: 캘린더 ui 수정

e3ea6e8

[fix]: ChallengeStatus에 TODAY추가

ebb2fdc

[fix]: ChallengeCalendarViewHolder에 TODAY 분기 처리 추가, 이미지 변경

2b70955

[refact]: 시간 관련 확장함수 리팩

f4da478

[feat]: visibility 관련 함수 추가

89d1834

[feat]: Apps 생성 팩토리 추가

b9d3d91

[refact]: challengeFragment 함수 쪼개기 등 refactoring, 뷰모델 추가

b170b59

[feat]: 챌린지 뷰 ui 수정

8610c41

[feat]: 네비게이션 아이콘 수정

7ee2014

[feat]: 네비게이션 아이콘 수정

7eab49a

[feat]: 캘린더 ui 일자 별 디테일 수정

04d6d67

[feat]: 리스트 편집 시 쓰레기 통 모양 아이콘 추가

e11aa10

[feat]: 삭제 아이콘 빨간색으로 색상 변경

5423691

[fix]: 다이얼로그 디테일 변경

d524fa8

jihyun0v0 added 💻feat 새로운 기능 추가 🐥지현 지현이 작업 labels Mar 15, 2024

jihyun0v0 requested review from kangyuri1114 and kez-lab March 15, 2024 13:14

jihyun0v0 self-assigned this Mar 15, 2024

jihyun0v0 requested a review from a team as a code owner March 15, 2024 13:14

jihyun0v0 removed request for kangyuri1114 and kez-lab March 15, 2024 13:16

jihyun0v0 added 2 commits March 15, 2024 22:47

[fix]: appAddViewModel 순서 수정

b97241d

[fix]: appAddViewModel init시 delay추가 하여 nullpointException 방지

1cc21c7

kangyuri1114 reviewed Mar 23, 2024

View reviewed changes

jihyun0v0 changed the base branch from develop to feat/mypage_ui March 23, 2024 11:58

jihyun0v0 changed the base branch from feat/mypage_ui to feat/usage_statics_ui March 23, 2024 11:58

jihyun0v0 changed the base branch from feat/usage_statics_ui to feat/mypage_ui March 23, 2024 11:59

jihyun0v0 added 9 commits March 24, 2024 08:37

[delete]: 필요 없는 주석 제거

93169d8

[refact]: 주석 제거, 뷰모델에서 데이터 가져와서 bind하는 함수 collect~~StateAndBind~~ -> …

34f6dc4

…init~~ 으로 함수 명 변경

[refact]: ChallengeFragment와 함수명 변경 통일

5c6e79b

[refact]: mainState, challengeState collect 함수 viewmodel안에 생성

969a7aa

[refact]: ChallengeStatusViewHolder Adapter로부터 분리

343ff59

[fix]: todayIndex가 -1일 경우의 분기처리

f0a401c

[fix]: 챌린지 없는 경우에도 usageGoals나오도록 변경, TODO 작성

baec8b9

[delete]: 주석 삭제

1ea065d

[refact]: 함수 분리

8310b41

kangyuri1114 reviewed Mar 24, 2024

View reviewed changes

kangyuri1114 approved these changes Mar 25, 2024

View reviewed changes

kangyuri1114 reviewed Mar 28, 2024

View reviewed changes

jihyun0v0 added 4 commits March 29, 2024 16:19

[fix]: state 가 null일 경우 분기처리 후 delayc추가

1e8b0a1

[delete]: log삭제

3d70298

[refact]: point로고 core module로 옮기기

6dc79db

[feat]: content description추가

00548b6

jihyun0v0 merged commit 5c67825 into feat/mypage_ui Mar 30, 2024

jihyun0v0 deleted the feat/challenge_ui branch March 30, 2024 13:21

kez-lab pushed a commit that referenced this pull request Nov 20, 2024

Merge pull request #167 from Team-HMH/feat/challenge_ui

ae2fc97

[�feat/challenge_ui]: 챌린지 현황 뷰 ui 수정

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat/challenge_ui]: 챌린지 현황 뷰 ui 수정 #167

[feat/challenge_ui]: 챌린지 현황 뷰 ui 수정 #167

jihyun0v0 commented Mar 15, 2024 •

edited

Loading

kangyuri1114 left a comment

kangyuri1114 Mar 23, 2024

jihyun0v0 Mar 23, 2024

kangyuri1114 Mar 23, 2024

jihyun0v0 Mar 23, 2024

kangyuri1114 Mar 23, 2024

jihyun0v0 Mar 23, 2024

kangyuri1114 Mar 23, 2024

jihyun0v0 Mar 23, 2024 •

edited

Loading

kangyuri1114 Mar 24, 2024

kangyuri1114 Mar 23, 2024

jihyun0v0 Mar 24, 2024

kangyuri1114 Mar 23, 2024

kangyuri1114 commented Mar 23, 2024

kangyuri1114 Mar 24, 2024

kangyuri1114 Mar 24, 2024

kangyuri1114 Mar 24, 2024

kangyuri1114 Mar 28, 2024

jihyun0v0 Mar 29, 2024

		// private val _usageGoalsState = MutableStateFlow(UsageGoalsState())
		// val usageGoalsState = _usageGoalsState.asStateFlow()

		@@ -0,0 +1,94 @@
		<vector xmlns:android="http://schemas.android.com/apk/res/android"

[feat/challenge_ui]: 챌린지 현황 뷰 ui 수정 #167

[feat/challenge_ui]: 챌린지 현황 뷰 ui 수정 #167

Conversation

jihyun0v0 commented Mar 15, 2024 • edited Loading

개요

작업 사항

스크린샷(optional)

kangyuri1114 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

jihyun0v0 Mar 23, 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

kangyuri1114 commented Mar 23, 2024

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

jihyun0v0 commented Mar 15, 2024 •

edited

Loading

jihyun0v0 Mar 23, 2024 •

edited

Loading