Skip to content

feat: 토너먼트 위시 가져오기 폴더 칩 화면 정식 경로 통합 - #783

Merged
ychany merged 3 commits into
devfrom
feat/782-by-wish-folder-chip
Nov 2, 2026
Merged

ychany merged 3 commits into
devfrom
feat/782-by-wish-folder-chip

Conversation

@ychany

@ychany ychany commented Nov 2, 2026

Copy link
Copy Markdown
Member

작업 요약

  • 토너먼트 "내 위시에서 가져오기"(by-wish)를 폴더 칩 화면으로 교체합니다
  • 임시 경로 by-wish-v2 를 제거합니다
  • 기존 후보 담기 동작은 그대로 유지합니다

작업 세부 내용

머지 조건

화면 교체

항목 이전 by-wish 이후
폴더 칩 없음 전체 · 내 폴더 목록. 칩을 고르면 그 폴더 위시만
카드 2열 WishSelectCard 위시 화면과 같은 3열 WishGrid
카운터 별도 헤더 N/담을 수 있는 수개 선택
위시 0개 TournamentErrorDialog 시안의 "가져올 위시가 없어요." 모달
목록 불러오기 마운트 때 전체 페이지 로드 무한 스크롤

유지한 규칙

  • READY 위시만 표시
  • 이미 후보에 담긴 상품 제외
  • 담을 위시가 없으면 "담을 수 있는 위시가 없어요" 안내
    • "위시가 모두 후보에 담겨 있어요." 설명은 전체 칩일 때만
      • 원인: 폴더 칩에서는 폴더가 비었을 수도 있음
  • 다음 은 기존 후보 담기 API · 성공 이동 · 에러 처리 그대로
  • 32개 초과 선택 시 토스트

칩 사이 선택 유지

  • 선택 상태에 위시 id 와 상품 id 를 함께 저장
    • 원인: 다른 칩으로 옮기면 이전 칩의 위시가 화면 목록에 없어 상품 id 를 찾을 수 없음
  • 여러 폴더에 있는 같은 위시는 위시 id 기준이라 1개로 셈

정리

대상 처리
by-wish-v2 페이지 · ByWishV2Content 삭제
FolderChipFilter · NoWishDialog · useFolderWishes by-wish 로 이관
WishSelectCard · WishSelectHeader · useWishSelection 삭제. 사용처 0곳
TOURNAMENT_ADD_ITEM_BY_WISH_V2 · getRouteType v2 경로 삭제

시안과 다른 점

테스트

  • e2e tournamentItemAdd 에 2개 추가
    • 칩을 오가며 고른 위시를 함께 담기 · 이미 후보에 담긴 상품 제외
  • 기존 담기 시나리오 통과 (폴더 목록 목만 추가)
  • 위시 · 토너먼트 · 옛 위시 화면 · 홈 e2e 81개 통과

스크린샷

선택 전 선택 중 폴더 칩 위시 없음

연관 이슈

closes #782

@ychany ychany added feature New feature or request WEB labels Nov 2, 2026
@vercel
vercel Bot temporarily deployed to Preview – piki-notify November 2, 2026 10:53 Inactive
@vercel

vercel Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated
piki Ready Ready Preview Nov 2, 2026 11:53am UTC
1 Skipped Deployment
Project Deployment Actions Updated
piki-notify Skipped Skipped Nov 2, 2026 11:53am UTC

@ychany ychany self-assigned this Nov 2, 2026
@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 26 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Repository UI
  • Review profile: CHILL
  • Plan: Advanced
  • Run ID: f9f29be5-d018-4ab5-ba3b-b1bfeeec1244
Commits

Reviewing files that changed from the base of the PR and between 71b100c and baa0374.

Files selected for processing: 13
  • apps/web/e2e/specs/tournament/tournamentItemAdd.spec.ts
  • apps/web/src/app/tournament/[id]/create/by-wish-v2/_components/ByWishV2Content.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish-v2/page.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish/_components/ByWishContent.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish/_components/FolderChipFilter.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish/_components/NoWishDialog.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish/_components/WishSelectCard.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish/_components/WishSelectHeader.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish/_hooks/useFolderWishes.ts
  • apps/web/src/app/tournament/[id]/create/by-wish/_hooks/useWishSelection.ts
  • apps/web/src/consts/route.ts
  • apps/web/src/utils/getRouteType.test.ts
  • apps/web/src/utils/getRouteType.ts
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@ychany
ychany force-pushed the feat/782-by-wish-folder-chip branch from fe9d1e0 to 3e45d3c Compare November 2, 2026 11:14
@vercel
vercel Bot temporarily deployed to Preview – piki-notify November 2, 2026 11:14 Inactive
@ychany
ychany force-pushed the feat/733-cleanup branch 2 times, most recently from f855464 to 19474e1 Compare November 2, 2026 11:48
Base automatically changed from feat/733-cleanup to dev November 2, 2026 11:51
- by-wish 화면에 폴더 칩 · 위시 그리드 · 위시 없음 모달 적용
  - 폴더 칩 컴포넌트 · 모달 · 폴더별 위시 훅을 by-wish-v2 에서 이관
- 기존 규칙 유지
  - READY 위시만 표시 · 이미 후보에 담긴 상품 제외
  - 다음 버튼은 기존 후보 담기 API 와 성공 · 에러 처리 그대로
- 선택 상태에 위시 id 와 상품 id 를 함께 저장
  - 원인: 칩을 오가며 고른 위시도 함께 담아야 함
- 쓰지 않게 된 WishSelectCard · WishSelectHeader · useWishSelection 삭제
- by-wish-v2 페이지 · TOURNAMENT_ADD_ITEM_BY_WISH_V2 삭제
- getRouteType 의 v2 경로와 테스트 정리
- 칩을 오가며 고른 위시를 함께 담기
- 이미 후보에 담긴 상품 제외
- 기존 담기 시나리오에 폴더 목록 목 추가
@ychany
ychany force-pushed the feat/782-by-wish-folder-chip branch from 3e45d3c to baa0374 Compare November 2, 2026 11:52
@vercel
vercel Bot temporarily deployed to Preview – piki-notify November 2, 2026 11:52 Inactive
@ychany
ychany merged commit 0a82e76 into dev Nov 2, 2026
7 checks passed
@ychany
ychany deleted the feat/782-by-wish-folder-chip branch November 2, 2026 11:54

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – piki — baa03740 Deployed Nov 2, 2026 by vercel[bot]
Preview – piki-notify — baa03740 Deployed Nov 2, 2026 by vercel[bot]
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature New feature or request WEB

Projects

None yet

Development

Successfully merging this pull request may close these issues.

토너먼트 위시 가져오기 폴더 칩 화면 정식 경로 통합

1 participant