Skip to content

feat: 토너먼트 폴더 칩 API 연결 및 위시 목 데이터 제거 (4단계) - #780

Merged
ychany merged 2 commits into
devfrom
feat/733-cleanup
Nov 2, 2026
Merged

ychany merged 2 commits into
devfrom
feat/733-cleanup

Conversation

@ychany

@ychany ychany commented Oct 30, 2026 •

Copy link
Copy Markdown
Member

작업 요약

  • 토너먼트 "내 위시에서 가져오기"(by-wish-v2) 폴더 칩을 폴더 · 위시 API 로 연결합니다
  • 위시 화면의 목 데이터와 목 전용 타입을 삭제합니다
  • 위시·폴더 화면 실제 API 전환 #733 API 연동 4개 PR 중 마지막입니다

작업 세부 내용

머지 조건

  • base 는 feat: 폴더 상세 폴더 API 연결 (3단계) #779 브랜치(feat/733-folder-detail). 이 PR diff 에는 4번째 작업만 보임
  • 앞 PR 머지 후 dev 위로 rebase 하고 base 를 dev 로 변경
  • 서버 폴더 API dev 반영 완료. dev 서버로 실제 흐름 확인하고 머지

토너먼트 폴더 칩

  • 폴더 칩을 폴더 목록 조회로 교체
  • 칩을 고르면 folderId 목록 조회. 전체 칩은 전체 위시 목록
    • 칩 전환 중에는 이전 목록 유지(keepPreviousData)
      • 원인: Suspense 쿼리로 바꾸면 칩을 누를 때마다 화면 전체가 로딩으로 바뀜
  • 위시 0개 안내는 전체 위시 기준
  • 무한 스크롤 추가
  • 범위: 다음 버튼의 후보 담기 연결은 제외
    • 원인: by-wish 와 by-wish-v2 를 합치는 작업에서 처리 (TOURNAMENT_ADD_ITEM_BY_WISH_V2 TEMP 경로)

목 데이터 제거

대상 내용
app/wish/_common/_consts/wishMock.ts 위시 12개 · 폴더 5개 목
public/mock/wish/*.jpg 목 이미지 6장
WishFolderT 목 폴더 타입. FolderT 로 대체
WishListItemT 목 위시 타입. GetWishlistResponseT 로 대체
  • 위시 · 폴더 화면의 목 참조 0곳

#733 인수 조건 점검

인수 조건 결과
새 화면에 목 데이터가 남지 않음 충족. 목 파일 · 이미지 · 타입 삭제
생성 · 삭제 · 이동 후 관련 목록 캐시 갱신 충족. 위시 목록 · 폴더 목록을 같이 갱신
에러 처리는 정책 문서를 따름 충족. 4xx 개별 안내, 401 · 5xx · 네트워크 전역
기존 getWishlist 를 덮어쓰지 않음 충족. 새 getWishes 추가

테스트

  • 스택 전체 기준 위시 · 토너먼트 · 옛 위시 화면 · 알림 e2e 78개 통과

스크린샷

화면 변화 없음

연관 이슈

closes #733

@ychany ychany added feature New feature or request WEB labels Oct 30, 2026
@ychany ychany self-assigned this Oct 30, 2026
@vercel

vercel Bot commented Oct 30, 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:49am UTC
1 Skipped Deployment
Project Deployment Actions Updated
piki-notify Skipped Skipped Nov 2, 2026 11:49am UTC

@coderabbitai

coderabbitai Bot commented Oct 30, 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 30 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: 2dabddbd-3734-4917-b6e1-40732bfb9f37
Commits

Reviewing files that changed from the base of the PR and between 437b1fa and 19474e1.

Files ignored due to path filters: 6
  • apps/web/public/mock/wish/wish-1.jpg is excluded by !**/*.jpg
  • apps/web/public/mock/wish/wish-2.jpg is excluded by !**/*.jpg
  • apps/web/public/mock/wish/wish-3.jpg is excluded by !**/*.jpg
  • apps/web/public/mock/wish/wish-4.jpg is excluded by !**/*.jpg
  • apps/web/public/mock/wish/wish-5.jpg is excluded by !**/*.jpg
  • apps/web/public/mock/wish/wish-6.jpg is excluded by !**/*.jpg
Files selected for processing: 5
  • apps/web/src/app/tournament/[id]/create/by-wish-v2/_components/ByWishV2Content.tsx
  • apps/web/src/app/tournament/[id]/create/by-wish-v2/_hooks/useFolderWishes.ts
  • apps/web/src/app/wish/_common/_consts/wishMock.ts
  • apps/web/src/app/wish/_common/_types/wish.ts
  • apps/web/src/types/wish.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 commented Oct 30, 2026

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Oct 30, 2026 •

Copy link
Copy Markdown
⚠️ Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

@ychany

ychany commented Nov 2, 2026

Copy link
Copy Markdown
Member Author

@coderabbitai review

@coderabbitai

coderabbitai Bot commented Nov 2, 2026 •

Copy link
Copy Markdown
Action not completed

Review rate limited.

Note: CodeRabbit is an incremental review system and does not re-review already reviewed commits. This command is applicable only when automatic reviews are paused.

- 폴더 칩을 폴더 목록 조회 API 로 교체
- 칩을 고르면 folderId 목록 조회로 교체
  - 칩 전환 중에는 이전 목록 유지
    - 원인: Suspense 로 화면 전체가 깜빡임
- 위시 0개 안내는 전체 위시 기준
- 무한 스크롤 추가
- wishMock.ts 와 public/mock/wish 이미지 6장 삭제
- 목 전용 타입 WishFolderT · WishListItemT 삭제
  - 원인: 위시 · 폴더 화면 전부 API 응답 타입 사용
@vercel
vercel Bot temporarily deployed to Preview – piki-notify November 2, 2026 11:48 Inactive
@ychany
ychany merged commit 71b100c into dev Nov 2, 2026
7 checks passed
@ychany
ychany deleted the feat/733-cleanup branch November 2, 2026 11:51

This branch was successfully deployed

1 active and 1 inactive deployments
Preview – piki — 19474e1e Deployed Nov 2, 2026 by vercel[bot]
Preview – piki-notify — 19474e1e 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.

위시·폴더 화면 실제 API 전환

1 participant