Skip to content

feat: 디자인 수정 및 api 응답 처리로직 수정 - #77

Merged
dasosann merged 9 commits into
mainfrom
feat/QA
Sep 14, 2026
Merged

dasosann merged 9 commits into
mainfrom
feat/QA

Conversation

@dasosann

@dasosann dasosann commented Sep 14, 2026

Copy link
Copy Markdown
Contributor

Greptile Summary

결제 상품의 수량 선택과 총액 계산을 추가하고, 관리자 주문 목록의 페이징 응답 처리 및 WebSocket 캐시 구조를 통일했습니다. 룰렛 참여 가능 상태와 이미지 리소스, Service Worker 캐시 전략을 수정했으며 여러 화면의 Tailwind 클래스를 디자인 토큰에 맞게 정리했습니다.

  • 관리자 주문을 모든 페이지에서 조회하고 hydration·WebSocket 캐시를 AdminOrder[]로 통일
  • 옵션권·뽑기권 구매 수량 선택 및 구매 API의 quantity 전달 지원
  • 무료·스페셜 룰렛 참여 상태, 안내 문구, 이미지 및 동적 렌더링 처리 개선
  • navigation 요청에 Network First 전략을 적용하고 성공 응답만 캐시
  • 전반적인 간격, 크기, typography 및 radius 클래스 정규화

Confidence Score: 5/5

현재 코드에서 병합을 막을 새로운 결함이나 미해결 이전 지적 사항이 확인되지 않아 안전하게 병합할 수 있습니다.

이전에 지적된 룰렛 초기 상태, Service Worker 오류 응답 캐시, 관리자 주문 hydration 형태와 페이지 누락, 정의되지 않은 Tailwind 클래스 문제가 모두 현재 코드에서 수정되었으며 관련 스레드도 해결 처리되었습니다. 이전 리뷰 이후 변경된 typography 클래스에서도 새로운 동작 오류는 확인되지 않았습니다.

Important Files Changed

Filename Overview
hooks/admin/useAdminOrders.ts 관리자 주문의 페이징 응답을 끝까지 조회해 단일 AdminOrder[]로 반환하도록 변경했습니다.
app/adminpage/dashboard/page.tsx 서버 prefetch도 클라이언트 query와 같은 배열 형태와 페이지 순회 방식을 사용하도록 통일했습니다.
hooks/admin/useAdminOrderSocket.ts 실시간 주문 생성·상태 변경 시 배열 기반 React Query 캐시를 직접 갱신하도록 수정했습니다.
components/common/charge-confirm/DirectChargeDrawer.tsx 수량 선택, 총 입금액 계산 및 구매 요청의 수량 전달을 추가했습니다.
components/common/charge-confirm/TossChargeDrawer.tsx 수량별 총 결제 금액을 표시하고 동일한 금액으로 Toss 송금을 실행하도록 변경했습니다.
hooks/usePurchaseProduct.ts 구매 mutation이 productId와 선택적 quantity를 받아 API query parameter로 전달하도록 확장되었습니다.
app/roulette/_components/ScreenRouletteMain.tsx 룰렛 상태가 확인되지 않은 동안 참여 가능 UI가 잘못 활성화되지 않도록 처리했습니다.
public/sw.js navigation 요청에 Network First 전략을 적용하고 정상 응답에 한해서만 캐시를 갱신하도록 개선했습니다.

Reviews (7): Last reviewed commit: "feat: 중복 style 제거" | Re-trigger Greptile

Comment thread app/roulette/_components/ScreenRouletteMain.tsx Outdated
Comment thread public/sw.js Outdated
Comment thread app/adminpage/dashboard/_components/AdminDashboard.tsx
Comment thread hooks/admin/useAdminOrders.ts Outdated
Comment thread app/main/_components/ProfileProgressCard.tsx Outdated
@dasosann
dasosann merged commit 009ac0b into main Sep 14, 2026
3 checks passed
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