Skip to content

Fix: stamp 현황판 수정#131

Merged
skyblue1232 merged 1 commit intodevelopfrom
api/#112/ai-video-api
Nov 15, 2025
Merged

Fix: stamp 현황판 수정#131
skyblue1232 merged 1 commit intodevelopfrom
api/#112/ai-video-api

Conversation

@skyblue1232
Copy link
Copy Markdown
Contributor

@skyblue1232 skyblue1232 commented Nov 15, 2025

🔥 작업 내용

  • stamp 현황판 수정
  • 에러처리 변경

🤔 추후 작업 사항

  • ex) 소셜 로그인 연동 (카카오, 구글)

🔗 이슈

  • close #

PR Point (To Reviewer)

  • ex) 로그인 입력 검증 로직 적절한지 확인 부탁드립니다.

📸 피그마 스크린샷 or 기능 GIF

(작업 내역 스크린샷)

Summary by CodeRabbit

릴리스 노트

  • Bug Fixes

    • 스탬프 로드 실패 시 사용자 경험 개선. 에러 메시지 대신 시각적 스탬프 보드 플레이스홀더를 표시합니다.
  • Refactor

    • 스탬프 상태 조회 API 통신 최적화로 안정성을 강화했습니다.

@skyblue1232 skyblue1232 self-assigned this Nov 15, 2025
@skyblue1232 skyblue1232 added the api api 연결 label Nov 15, 2025
@skyblue1232 skyblue1232 added the fix 버그 및 자잘한 오류 수정 label Nov 15, 2025
@vercel
Copy link
Copy Markdown

vercel bot commented Nov 15, 2025

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

Project Deployment Preview Comments Updated (UTC)
af-fe Ready Ready Preview Comment Nov 15, 2025 6:45am

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Nov 15, 2025

Walkthrough

메인 페이지의 에러 상태에서 기존 오류 메시지 텍스트를 StampBoard 컴포넌트(count={0}, total={10})로 대체했습니다. 동시에 getStampStatus API 호출 시 사용되는 API 클라이언트를 apiAuth에서 apiWithToken으로 변경했습니다.

Changes

Cohort / File(s) Summary
메인 페이지 에러 상태 UI 업데이트
src/pages/main/index.tsx
isError 조건부 렌더링 시 기존 오류 메시지 문단을 <StampBoard count={0} total={10} /> 컴포넌트로 변경
스탬프 상태 조회 API 클라이언트 수정
src/shared/main/api/getStampStatus.ts
API 클라이언트를 apiAuth에서 apiWithToken으로 변경; 관련 import 업데이트

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • 주의 사항:
    • apiWithToken 변경이 인증 흐름에 미치는 영향 확인 필요
    • StampBoard 컴포넌트의 count={0}, total={10} 기본값이 의도한 에러 상태 UI인지 검증

Possibly related PRs

Suggested reviewers

  • jjangminii
  • KongMezu

Poem

🐰 오류 무덤에 스탬프를 심고,
토큰 향기로 API를 깨우니,
에러 메시지는 사라지고
예쁜 보드만 남아있네~
한 줄의 변화가
사용자 경험을 바꾸는 마법! ✨

Pre-merge checks and finishing touches

❌ Failed checks (1 warning, 1 inconclusive)
Check name Status Explanation Resolution
Docstring Coverage ⚠️ Warning Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%. You can run @coderabbitai generate docstrings to improve docstring coverage.
Description check ❓ Inconclusive 기본 구조는 템플릿을 따르고 있으나 여러 필수 섹션이 미완성 상태입니다. 이슈 번호 추가, PR Point에 실제 검토 사항 작성, 스크린샷/GIF 첨부가 필요합니다.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed 제목이 주요 변경 사항(stamp 현황판 수정 및 에러 처리 변경)을 명확하게 요약하고 있습니다.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch api/#112/ai-video-api

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 830288d and 5c72ef9.

📒 Files selected for processing (2)
  • src/pages/main/index.tsx (1 hunks)
  • src/shared/main/api/getStampStatus.ts (2 hunks)
🧰 Additional context used
🧬 Code graph analysis (2)
src/pages/main/index.tsx (2)
src/shared/components/main/components/stampBoard/StampBoard.tsx (3)
  • StampBoardProps (11-51)
  • _ (38-45)
  • StampBoardProps (5-9)
src/shared/components/main/components/stampBoard/Stamp.tsx (1)
  • StampProps (10-35)
src/shared/main/api/getStampStatus.ts (2)
src/shared/api/instance.ts (1)
  • apiWithToken (14-17)
src/shared/types/authtypes.ts (1)
  • ApiResponse (1-6)
⏰ 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). (1)
  • GitHub Check: build-and-deploy
🔇 Additional comments (1)
src/shared/main/api/getStampStatus.ts (1)

1-1: API 클라이언트 변경이 올바릅니다.

검증 결과, apiWithToken으로의 변경이 정확합니다. apiWithToken은 토큰이 필요한 요청용 인스턴스로 설계되었으며, Bearer 토큰을 자동으로 첨부하고 401 응답 시 토큰을 자동으로 갱신합니다. /api/stamps/collection 엔드포인트는 인증이 필요하므로 이 변경이 적절하며, 코드베이스의 다른 인증된 API 호출들(fetchMyPage, fetchMyInfo, getPostCardDetail, postRecommendCourse, stampAcquire 등)과 일관된 패턴입니다.


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.

@github-actions github-actions bot added the comment 필요한 주석 추가 및 변경 label Nov 15, 2025
@github-actions
Copy link
Copy Markdown

🏷️ Labeler has automatically applied labels based on your PR title, branch name, or commit message.
Please verify that they are correct before merging.

Copy link
Copy Markdown
Contributor

@KongMezu KongMezu left a comment

Choose a reason for hiding this comment

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

수고하셨습니다

@skyblue1232 skyblue1232 merged commit d226a85 into develop Nov 15, 2025
7 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api api 연결 comment 필요한 주석 추가 및 변경 fix 버그 및 자잘한 오류 수정

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants