[FEAT/#35] 루틴 챌린지 및 Todo 미션 컴포넌트 구현 by hyeminililo · Pull Request #72 · TEAM-Cherrish/Cherrish-Android

hyeminililo · 2026-01-16T11:49:36Z

Related issue 🛠

closed [FEAT] 챌린지 뷰 컴포넌트 구현 #35

Work Description ✏️

루틴 챌린지 온보딩 바디 부분 구현
루틴 Todo 챌린지 온보딩 바디 부분 구현
ktLint 실행

Screenshot 📸

루틴 챌린지 온보딩 바디

Screen_recording_20260116_180228.mp4

Todo 챌린지 미션

Screen_recording_20260116_204241.mp4

Uncompleted Tasks 😅

N/A

To Reviewers 📢

중간에 브랜치를 바꿔서 다시 적용을 해서 커밋명 순서가 조금 다를 수 있습니다 빠른 피드백 환영입니다유 ~~~☺️

Summary by CodeRabbit

릴리스 노트

New Features
- 루틴 선택 온보딩 화면 추가 — 2열 칩 그리드로 항목 선택 가능
- 미션 선택 온보딩 화면 추가 — 목록형 칩으로 항목 선택/토글 지원
- 선택 상태 관리를 위한 UI 상태 및 데이터 모델 추가 (루틴·미션 모델 및 UI 상태)
Style
- 미션 카드 칩의 가로 간격 소폭(6→8dp) 조정으로 레이아웃 정렬 개선

_{✏️ Tip: You can customize this high-level summary in your review settings.}

- ChallengeMissionCardChip LazyColumn으로 수정

- ktLint 실행 - isSelected = true인 상태에서 한 번 더 눌렀을 떄 원래로 바뀌도록 구현

coderabbitai · 2026-01-16T11:49:47Z

Walkthrough

루틴·미션 선택용 불변 모델과 UI 상태를 추가하고, 두 개의 신규 Composable(ChallengeRoutineOnboardingBody, ChallengeMissionOnboardingBody)을 도입했습니다. 기존 ChallengeMissionCardChip의 Row 간격이 6.dp→8.dp로 조정되고 Preview 변수명이 selected→isSelected로 변경되었습니다.

Changes

Cohort / File(s)	Summary
데이터 모델 `app/src/main/java/com/cherrish/android/presentation/challenge/mission/model/ChallengeMissionModel.kt`, `app/src/main/java/com/cherrish/android/presentation/challenge/routine/model/ChallengeRoutineModel.kt`	신규 불변 데이터 클래스 추가: `ChallengeMissionModel(id: Int, missionContent: String, isSelected: Boolean)` 및 `ChallengeRoutineModel(id: Long, routine: String, isSelected: Boolean)` (`@Immutable`)
UI 상태 `app/src/main/java/com/cherrish/android/presentation/challenge/mission/ChallengeMissionUiState.kt`, `app/src/main/java/com/cherrish/android/presentation/challenge/routine/ChallengeRoutineUiState.kt`	신규 UI 상태 추가: `ChallengeMissionUiState(missions: ImmutableList<...>)` 및 `ChallengeRoutineUiState(routine: String, isSelected: Boolean)` (FakeRoutine 포함)
온보딩 컴포넌트 `app/src/main/java/com/cherrish/android/presentation/challenge/component/ChallengeRoutineOnboardingBody.kt`, `app/src/main/java/com/cherrish/android/presentation/challenge/component/ChallengeMissionOnboardingBody.kt`	신규 Composable 추가: 루틴은 2열 `LazyVerticalGrid`로 표시되는 선택형 칩 목록, 미션은 `LazyColumn`의 선택형 항목 목록. 각각 `onItemClick` 콜백 및 Preview 포함
기존 UI 컴포넌트 변경 `app/src/main/java/com/cherrish/android/presentation/challenge/component/ChallengeMissionCardChip.kt`	Row `horizontalArrangement` 6.dp → 8.dp, Preview 변수명 `selected` → `isSelected` 및 onClick 연결 업데이트

Sequence Diagram(s)

sequenceDiagram
    participant User as User
    participant RoutineBody as ChallengeRoutineOnboardingBody
    participant MissionBody as ChallengeMissionOnboardingBody
    participant Callback as onItemClick
    participant State as UIState

    User->>RoutineBody: 루틴 칩 클릭
    RoutineBody->>Callback: onItemClick(itemId)
    Callback->>State: 선택 상태 업데이트
    State->>RoutineBody: recomposition (updated items)
    RoutineBody->>User: 렌더링 갱신

    User->>MissionBody: 미션 항목 클릭
    MissionBody->>Callback: onItemClick(itemId)
    Callback->>State: 선택 상태 업데이트
    State->>MissionBody: recomposition (updated items)
    MissionBody->>User: 렌더링 갱신

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

[Feat/#57] 챌린지 Todo 미션칩 컴포넌트 구현 #58: ChallengeMissionCardChip 관련 미세 UI 변경(간격·Preview 변수명)과 직접 연관됨.
[MOD/#29] 공통선택칩 컴포넌트 코드리뷰 반영 #31: 칩 컴포넌트 API 리팩터 및 사용 방식 변경과 연관 가능성 있음.
[DEL/#69] 미션카드 칩 컴포넌트 삭제 #70: 온보딩 컴포넌트 및 미션/칩 UI 추가와 연관될 가능성 있음.

Suggested reviewers

sohee6989
nhyeonii
usuuhyn

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	PR 제목이 주요 변경사항을 명확하게 요약하고 있습니다. 루틴 챌린지와 Todo 미션 컴포넌트 구현이라는 핵심 내용을 담고 있습니다.
Description check	✅ Passed	PR 설명이 템플릿 구조를 따르고 있으며, 관련 이슈, 작업 내용, 스크린샷, 완료 상태가 모두 포함되어 있습니다.
Linked Issues check	✅ Passed	이슈 `#35의` 요구사항 중 TODO 미션 선택 컴포넌트와 챌린지 바디 부분 컴포넌트가 구현되었으며, 루틴 챌린지 선택 컴포넌트는 미완료 상태입니다.
Out of Scope Changes check	✅ Passed	모든 코드 변경사항이 이슈 `#35의` 요구사항 범위 내에 있으며, 새로운 UI 컴포넌트와 데이터 모델이 일관되게 추가되었습니다.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing touches

📝 Generate docstrings

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

sohee6989

수고하셨습니다!!

sohee6989 · 2026-01-17T13:14:00Z

+
+@Composable
+fun ChallengeMissionOnboardingBody(
+    items: List<ChallengeMissionUiState>,


p1: immutablelist?!

추가로 여기에 바로 uiState를 넘겨주기보다는 스크린 상위에서 한번만 uiState를 넘겨주는 방향이 더 맞을 것 같아요 이 컴포넌트 필요한 값만 내려주는 형태가 사용하지 않는 uiState의 정보들을 가지고 있을 필요가 없기 때문에 더 적합할 것 같습니다 !

넵 두 리뷰 다 수정해서 리팩토링 진행하겠습니다 !

sohee6989 · 2026-01-17T13:14:30Z

+            verticalArrangement = Arrangement.spacedBy(space = 8.dp),
+            modifier = Modifier.fillMaxWidth()
+        ) {
+            itemsIndexed(items = items) { index, item ->


p1: key값 활용하는거 어떠세요?!

넵 반영해서 수정하겠습니다 !

sohee6989 · 2026-01-17T13:15:17Z

+
+@Composable
+fun ChallengeRoutineOnboardingBody(
+    items: List<ChallengeRoutineUiState>,


p1: immutablelist?!

여기도 uiState 전체 보다는 필요한 정보들만 받을 수 잇도록 수정해주세요 ~~

sohee6989 · 2026-01-18T00:54:22Z

@@ -0,0 +1,6 @@
+package com.cherrish.android.presentation.challenge.routine.model
+


P1:immutable?!

아니 점점 줄임말로 말해주시네요 감사합니다아 ~ 😙

sohee6989 · 2026-01-18T00:54:39Z

+@Immutable
+data class ChallengeRoutineUiState(
+    val routine: String = "",
+


p2: 공백 없애주자용~!!

아아 넵넵 ! !

sohee6989 · 2026-01-18T00:54:54Z

@@ -0,0 +1,6 @@
+package com.cherrish.android.presentation.challenge.mission.model
+


p1: Immutable?!

Aha ! 넵 ~~~!!

sohee6989 · 2026-01-18T00:55:22Z

+)
+
+data class ChallengeMissionScreenUiState(
+    val missions: List<ChallengeMissionUiState>


p1: immutableList?!

수정했습니다아 ~

sohee6989 · 2026-01-18T00:57:23Z

+    val isSelected: Boolean = false
+)
+
+data class ChallengeMissionScreenUiState(


p1: immutable?!

바꾸겠습니다아 ~~!!

sohee6989 · 2026-01-18T00:57:29Z

+
+import com.cherrish.android.presentation.challenge.mission.model.ChallengeMissionModel
+
+data class ChallengeMissionUiState(


p1: immutable?!

이 부분도 수정하겠습니다아 ~~

nhyeonii

수고하셨습ㄴㅣ다 ~~ 리뷰 확인 후 반영 부탁드릴게요!

nhyeonii · 2026-01-18T07:14:29Z

+
+@Composable
+fun ChallengeMissionOnboardingBody(
+    items: List<ChallengeMissionUiState>,


추가로 여기에 바로 uiState를 넘겨주기보다는 스크린 상위에서 한번만 uiState를 넘겨주는 방향이 더 맞을 것 같아요 이 컴포넌트 필요한 값만 내려주는 형태가 사용하지 않는 uiState의 정보들을 가지고 있을 필요가 없기 때문에 더 적합할 것 같습니다 !

nhyeonii · 2026-01-18T07:17:54Z

+
+@Composable
+fun ChallengeRoutineOnboardingBody(
+    items: List<ChallengeRoutineUiState>,


여기도 uiState 전체 보다는 필요한 정보들만 받을 수 잇도록 수정해주세요 ~~

nhyeonii · 2026-01-18T07:19:35Z

+            verticalArrangement = Arrangement.spacedBy(space = 12.dp),
+            modifier = Modifier.fillMaxWidth()
+        ) {
+            itemsIndexed(items) { index, item ->


P1 : 여기도 key 값 지정해주셍용 ~~

넵 수정하겠습니다 !

nhyeonii · 2026-01-18T07:27:02Z

+
+import com.cherrish.android.presentation.challenge.mission.model.ChallengeMissionModel
+
+data class ChallengeMissionUiState(


P1 : UiState는 화면 당 하나만 나타나도록 수정해주세요! 화면 상태가 여러 UiState로 분산되면 흐름을 파악하기 어렵고 상태 동기화 비용이 커질 수 있어요!

넵 하나의 State로 묶을게요!!

- ChallengeMissionUiState 내 선택한 미션을 관리하는 missions는 불필요하고 뷰모델에서 처리할 수 있을 것 같아 삭제

nhyeonii

고생햇슈 ~~

hyeminililo added 12 commits January 16, 2026 17:47

[REF/#35] ChallengeMissionCardChip 수정

fcaea62

- ChallengeMissionCardChip LazyColumn으로 수정

[FEAT/#35] ChallengeRoutineModel 구현

e744e38

[FEAT/#35] ChallengeRoutineUiState 구현

42132ef

[FEAT/#35] ChallengeRoutineMissionModel 구현

693dcb6

[FEAT/#35] ChallengeMissionUiState 구현

b321716

[REP/#35] ChallengeMissionModel 함수명 변경

c082b82

[FEAT/#35] ChallengeRoutineOnboarding 구현

a05a627

[FEAT/#35] ChallengeRoutineMissionOnboarding 구현

8e68e3a

[REP/#35] ChallengeMissionModel 함수명 변경

372c221

[REP/#35] ChallengeMissionModel 함수명 변경

b887a90

[ADD/#35] ChallengeRoutineOnboadingBody 터치 수정

41741ca

[ADD/#35] 선택 더블 클릭시 해제하면 색 변경하는 코드 추가

bd90062

- ktLint 실행 - isSelected = true인 상태에서 한 번 더 눌렀을 떄 원래로 바뀌도록 구현

hyeminililo requested a review from a team as a code owner January 16, 2026 11:49

hyeminililo self-assigned this Jan 16, 2026

hyeminililo added FEAT✨ 새로운 기능 구현 혜민🍒 혜민 담당 labels Jan 16, 2026

sohee6989 approved these changes Jan 18, 2026

View reviewed changes

nhyeonii requested changes Jan 18, 2026

View reviewed changes

hyeminililo added 8 commits January 18, 2026 21:12

[REF/#35] 코드리뷰 기반으로 반영 리팩토링

d6d97f5

[CHORE/#35] ktLint 실행

d64d06e

[CHORE/#35] ktLint 실행

5622ea5

[REF/#35] 코드리뷰 기반으로 리팩토링 진행

f0db05f

[CHORE/#35] ktlInt 실행

d66d8c9

[REF/#45] 코드리뷰 반영 안 된 부분 반영

7920b8e

[REF/#35] 코드리뷰 반영 안 된 부분 반영

a3e1d16

[DEL/#35] ChallengeMissionUiState 변수 삭제

f05d736

- ChallengeMissionUiState 내 선택한 미션을 관리하는 missions는 불필요하고 뷰모델에서 처리할 수 있을 것 같아 삭제

nhyeonii approved these changes Jan 18, 2026

View reviewed changes

[CHORE/#35] ktLint 실행

d2aea3e

hyeminililo merged commit e6dfa54 into develop Jan 18, 2026
3 checks passed

This was referenced Jan 19, 2026

[FEAT/#42] 챌린지 루틴 선택 뷰 스크린 구현 #91

Merged

[FEAT/#92] 챌린지 미션 뷰 컴포넌트 조립 #94

Merged

[FEAT/#103] 챌린지 루틴 목록 조회 API 연결 완료 #119

Closed

		@@ -0,0 +1,6 @@
		package com.cherrish.android.presentation.challenge.routine.model


		import com.cherrish.android.presentation.challenge.mission.model.ChallengeMissionModel

		data class ChallengeMissionUiState(

Conversation

hyeminililo commented Jan 16, 2026 • edited by coderabbitai Bot Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Related issue 🛠

Work Description ✏️

Screenshot 📸

Uncompleted Tasks 😅

To Reviewers 📢

Summary by CodeRabbit

릴리스 노트

Uh oh!

coderabbitai Bot commented Jan 16, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Walkthrough

Changes

Sequence Diagram(s)

Estimated code review effort

Possibly related PRs

Suggested reviewers

Uh oh!

sohee6989 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!

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!

nhyeonii 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!

nhyeonii left a comment

Choose a reason for hiding this comment

hyeminililo commented Jan 16, 2026 •

edited by coderabbitai Bot

Loading

coderabbitai Bot commented Jan 16, 2026 •

edited

Loading