Skip to content

[DEL/#69] 미션카드 칩 컴포넌트 삭제 #70

Merged
hyeminililo merged 3 commits intodevelopfrom
del/#69-missioncard-chip-component
Jan 15, 2026
Merged

[DEL/#69] 미션카드 칩 컴포넌트 삭제 #70
hyeminililo merged 3 commits intodevelopfrom
del/#69-missioncard-chip-component

Conversation

@hyeminililo
Copy link
Copy Markdown
Contributor

@hyeminililo hyeminililo commented Jan 15, 2026

Related issue 🛠

Work Description ✏️

  • 미션 카드 칩 컴포넌트 삭제
  • 기존 BaseChip, SelectionChip 구조 -> SelectionChip 병합
  • ktLint 실행

Screenshot 📸

삭제 관련 코드라 어떤 걸 올려야 할지 모르겠어요 ,,,

Uncompleted Tasks 😅

  • N/A

To Reviewers 📢

피곤하시지만 ,, 피드백 부탁드립니당 ,, 🫶🏻

Summary by CodeRabbit

릴리스 노트

  • Refactor
    • 선택 칩 컴포넌트의 내부 구조를 간소화하고 스타일/상호작용 로직을 통합했습니다.
    • 텍스트 스타일과 클릭 동작이 일관되게 적용되도록 변경했습니다.
  • Chore
    • 미션 카드 관련 컴포넌트를 제거했습니다.

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

- 불필요한 미션 칩 삭제
- ktLint 실행
- SelectionBase칩과 SelectionChip 합치기
- ktLint 실행
@hyeminililo hyeminililo self-assigned this Jan 15, 2026
@hyeminililo hyeminililo requested a review from a team as a code owner January 15, 2026 18:07
@hyeminililo hyeminililo added DELETE➖ 더 이상 사용하지 않는 파일, 코드, 리소스의 단순 제거 혜민🍒 혜민 담당 labels Jan 15, 2026
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Jan 15, 2026

Walkthrough

디자인 변경으로 인해 기본 선택 칩 컴포넌트가 제거되고, CherrishSelectionChip에 스타일/동작이 인라인으로 통합되었으며, 미션 칩 관련 파일이 삭제되었습니다.

Changes

코호트 / 파일(s) 변경 요약
기본 칩 컴포넌트 제거
app/src/main/java/com/cherrish/android/core/designsystem/component/chip/CherrishSelectionBaseChip.kt
최상위 Composable CherrishSelectionBaseChip 함수 삭제 (SELECTIONCHIP/MISSIONCARD 스타일 포함)
선택 칩 인라인화 및 리팩토링
app/src/main/java/com/cherrish/android/core/designsystem/component/chip/CherrishSelectionChip.kt
CherrishSelectionBaseChip 의존성 제거, Column 기반으로 배경/테두리/클릭 로직 인라인화, 텍스트 스타일 단일화 및 색상 계산 로컬 처리
미션 칩 컴포넌트 삭제
app/src/main/java/com/cherrish/android/core/designsystem/component/chip/CherrishSelectionMissionChip.kt
CherrishMissionCard 공개 함수 및 관련 Preview 함수 전체 삭제

Sequence Diagram(s)

(생성 조건 미충족 — 해당 섹션 생략)

Estimated code review effort

🎯 3 (Moderate) | ⏱️ ~20 minutes

Possibly related PRs

Suggested labels

REFACTOR💡

Suggested reviewers

  • nhyeonii
  • sohee6989
  • 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 제목이 주요 변경 사항을 명확하게 요약하고 있으며, 미션카드 칩 컴포넌트 삭제라는 주요 목표를 잘 나타냅니다.
Description check ✅ Passed PR 설명이 저장소의 템플릿 구조를 따르며 관련 이슈, 작업 내용, 스크린샷 섹션, 미완료 작업을 포함하고 있습니다.
Linked Issues check ✅ Passed PR 변경사항이 #69 이슈의 모든 요구사항을 충족합니다: 미션 칩 컴포넌트 삭제(CherrishSelectionMissionChip.kt 제거), 분기 처리 삭제(BaseChip를 SelectionChip로 병합), ktLint 실행.
Out of Scope Changes check ✅ Passed 모든 변경사항이 #69 이슈의 범위 내에 있습니다: 미션 칩 제거, 구조 통합, 코드 스타일 정리는 모두 이슈의 목표와 일치합니다.

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

✨ Finishing touches
  • 📝 Generate docstrings


📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 09f6e26 and f1347ab.

📒 Files selected for processing (1)
  • app/src/main/java/com/cherrish/android/core/designsystem/component/chip/CherrishSelectionChip.kt
🚧 Files skipped from review as they are similar to previous changes (1)
  • app/src/main/java/com/cherrish/android/core/designsystem/component/chip/CherrishSelectionChip.kt
⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (2)
  • GitHub Check: PR Build Check
  • GitHub Check: PR Lint Check

✏️ Tip: You can disable this entire section by setting review_details to false in your review settings.


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.

Copy link
Copy Markdown
Contributor

@nhyeonii nhyeonii left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

폰트 한번만 확인해주시고 머지해주세요 ~~~


val textColor = if (isSelected) CherrishTheme.colors.gray800 else CherrishTheme.colors.gray700

val textStyle = if (isSelected) {
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image

P1 : 이거 textStyle BodyM14 고정인것 같숨둥 !!

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

넵 수정했습니다 !!

- textStyle 통일
- ktLint 실행
@hyeminililo hyeminililo merged commit 8bead24 into develop Jan 15, 2026
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

DELETE➖ 더 이상 사용하지 않는 파일, 코드, 리소스의 단순 제거 혜민🍒 혜민 담당

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[DEL] 챌린지 컴포넌트 삭제

2 participants