Skip to content

[Fix/#83] 빌릴게 카테고리 필터가 목록을 실제로 거르지 않던 문제 수정 - #84

Closed
xeoxxn wants to merge 2 commits into
mainfrom
fix/#83-bililge-category-filter
Closed

xeoxxn wants to merge 2 commits into
mainfrom
fix/#83-bililge-category-filter

Conversation

@xeoxxn

@xeoxxn xeoxxn commented Sep 29, 2026 •

Copy link
Copy Markdown
Collaborator

#️⃣연관된 이슈

🎯 해결하려는 문제가 무엇인가요?

빌릴게 대여 목록 화면의 카테고리 필터 칩(전체/전자기기/생활잡화/상비약/위생용품)을 눌러도 선택 상태만 바뀌고 목록은 그대로였다.

❓ 왜 해결해야 하나요?

목업 데이터(BILILGE_ITEMS)에 카테고리 정보 자체가 없어서 필터링이 애초에 불가능한 구조였다(BililgeCategoryFilter에도 "카테고리·물품 매핑 API가 아직 없어 필터링 없이 선택 상태만 표시한다"는 주석이 있었음). 사용자가 카테고리를 눌러도 결과가 안 바뀌니 기능이 고장난 것처럼 보인다.

⭐ 어떻게 해결했나요?

  • (커밋 1) BililgeItem에 category: BililgeCategory 필드를 추가하고, 목업 데이터 19개 물품 전부에 분류를 채웠다. Figma에는 카테고리별 그룹핑이 따로 정의돼 있지 않아(필터 칩 UI만 존재) 물품명 기준으로 임의 분류했다.
  • (커밋 2) BililgeListScreen에서 선택된 category가 "전체"가 아니면 BILILGE_ITEMS를 item.category로 필터링하도록 연결했다. BililgeCategoryFilter의 "필터링 없이 선택 상태만 표시" 주석도 실제 동작에 맞게 갱신했다.

🧩 이 PR의 한계 & 트레이드오프

  • 카테고리 분류가 Figma나 API 기준이 아니라 물품명 기반 임의 판단이라, 실제 카테고리 분류 기준이 정해지면 조정이 필요할 수 있다.
  • 실 API 연동 전이라 계속 프런트 목업 데이터에 의존한다(기존 방식과 동일).

⛓️ 기존 기능에 미치는 영향

  • BililgeListScreen/BililgeCategoryFilter/bililgeItems.ts 외 영향 없음. BililgeItem.category가 새 필수 필드지만 이 파일들 밖에서 BililgeItem을 직접 생성하는 곳은 없어 타입 에러 없음.

🔀 Edge Case & 실패 시나리오

  • 지금 분류 기준으로는 모든 카테고리에 최소 1개 이상 물품이 있어 빈 목록 케이스는 없다. 추후 실 데이터 연동 시 빈 카테고리가 생기면 별도 empty state 처리가 필요할 수 있다.

📋 검토한 대안과 선택 이유

  • BililgeItem에 직접 category 필드를 추가하는 대신 별도 물품-카테고리 매핑 테이블도 고려했지만, 목업 데이터 자체가 배열 하나라 필드 하나 추가가 더 단순하고 실 API 응답 형태(보통 아이템 자체에 카테고리가 포함됨)에도 더 가깝다.

💬 리뷰 포인트

  • [c] 물품별 카테고리 분류(예: 밴드/알콜스왑을 상비약으로 묶은 것)가 실제 의도와 맞는지 확인 부탁드립니다.

Summary by CodeRabbit

  • New Features
    • Rental items are now organized by category, and selecting a category filters the list to matching items. Selecting “전체” shows all items.

xeoxxn and others added 2 commits September 29, 2026 10:56
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 5 <noreply@anthropic.com>
@coderabbitai

coderabbitai Bot commented Sep 29, 2026

Copy link
Copy Markdown

Review in Change Stack →

Navigate logical layers of code changes, visualize relationships, and explore their blast radius.

Note

Currently processing new changes in this PR. This may take a few minutes, please wait...

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Advanced

Run ID: 90f20d46-78c9-4f35-a3f3-466845f1a4c3

📥 Commits

Reviewing files that changed from the base of the PR and between d91fb49 and 133e062.

📒 Files selected for processing (3)
  • src/features/bililge/BililgeListScreen.tsx
  • src/features/bililge/components/BililgeCategoryFilter.tsx
  • src/features/bililge/constants/bililgeItems.ts
 ________________________________
< NVIDIA inside, Rabbit outside. >
 --------------------------------
  \
   \   \
        \ /\
        ( )
      .( o ).
✨ Finishing Touches
📝 Generate docstrings
  • Commit to this branch
  • Create a new PR
🧪 Generate unit tests (beta)
  • Commit to this branch
  • Create a new PR

Comment @coderabbitai help to get the list of available commands.

@xeoxxn

xeoxxn commented Sep 29, 2026

Copy link
Copy Markdown
Collaborator Author

빌릴게 QA 반영 수정 PR(#80)에 커밋을 옮겨 합쳤습니다. 그쪽에서 계속 진행합니다.

@xeoxxn xeoxxn closed this Sep 29, 2026
@xeoxxn
xeoxxn deleted the fix/#83-bililge-category-filter branch September 29, 2026 01:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

빌릴게 카테고리 필터가 목록을 실제로 거르지 않음

1 participant