Skip to content

[Fix/#141] 내 보드 생성 시 리다이렉트 수정#142

Merged
KwonDeaGeun merged 3 commits intodevelopfrom
fix/#141
Dec 21, 2025
Merged

[Fix/#141] 내 보드 생성 시 리다이렉트 수정#142
KwonDeaGeun merged 3 commits intodevelopfrom
fix/#141

Conversation

@KwonDeaGeun
Copy link
Copy Markdown
Contributor

@KwonDeaGeun KwonDeaGeun commented Dec 21, 2025

Summary

관련 있는 Issue를 태그해주세요. (e.g. > - #100)

해당 PR에 대한 작업 내용을 요약하여 작성해주세요.

Tasks

  • 해당 PR에 수행한 작업을 작성해주세요.
  • 내 보드 생성 시 리다이렉트 수정

Summary by CodeRabbit

  • 보안 개선
    • 완료 페이지에서 이동 전 사용자 인증 상태를 확인하여, 인증 결과에 따라 적절한 페이지로 이동합니다.
  • 사용성
    • 이동 버튼에 로딩 상태와 비활성화 동작을 적용하여 처리 중임을 시각적으로 표시합니다.
    • 인증 오류 발생 시 로그를 남기고 안전하게 홈으로 반환하는 폴백 동작을 추가했습니다.

✏️ Tip: You can customize this high-level summary in your review settings.

@KwonDeaGeun KwonDeaGeun self-assigned this Dec 21, 2025
@KwonDeaGeun KwonDeaGeun linked an issue Dec 21, 2025 that may be closed by this pull request
@vercel
Copy link
Copy Markdown

vercel bot commented Dec 21, 2025

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

Project Deployment Review Updated (UTC)
2025-seasonthon-team-80-fe Ready Ready Preview, Comment Dec 21, 2025 4:06pm

@coderabbitai
Copy link
Copy Markdown

coderabbitai bot commented Dec 21, 2025

Walkthrough

join-complete-page의 버튼 핸들러를 비동기화하고 useAuthStore.checkAuth() 결과에 따라 ROUTES.BOARD 또는 ROUTES.HOME로 조건부 네비게이션하도록 변경했습니다. 로딩 상태와 예외 처리(log)도 추가되었습니다.

Changes

Cohort / File(s) 변경 요약
인증 기반 조건부 네비게이션
src/pages/joinPage/pages/join-complete-page.tsx
useAuthStore 임포트, handleClickasync로 변경해 await checkAuth() 수행. 인증 성공 시 ROUTES.BOARD, 실패 또는 오류 시 ROUTES.HOME로 분기. 로딩 상태와 버튼 비활성/레이블 변경, 오류 로그 추가.

Sequence Diagram(s)

sequenceDiagram
    participant UI as JoinCompletePage (component)
    participant Auth as useAuthStore.checkAuth()
    participant Router as Router/Navigation

    UI->>Auth: call checkAuth() (await)
    Auth-->>UI: returns true / false or throws
    alt auth true
        UI->>Router: navigate(ROUTES.BOARD)
    else auth false / error
        UI->>Router: navigate(ROUTES.HOME)
    end
Loading

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~5-10 minutes

  • 검토 포인트:
    • checkAuth()의 반환값(타입) 및 예외 처리 방식 확인
    • 로딩 상태가 버튼/화면에 정확히 반영되는지 확인
    • 네비게이션 분기에서 null/undefined 케이스 처리 여부

Possibly related issues

Possibly related PRs

Suggested labels

🛠 Fix

Suggested reviewers

  • hyesngy

Poem

🐰 버튼을 톡, 마음은 폴짝,
체크해봐, 문이 열릴까?
맞으면 BOARD로 깡충, 아니면 HOME으로 살금,
로딩등 켜고 기다리는 토끼의 춤,
배포 당근 하나, 냠냠!

Pre-merge checks and finishing touches

❌ Failed checks (2 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명이 템플릿 구조를 따르지만 필수 정보가 부족합니다. Summary 섹션에서 Issue 태그와 작업 내용 설명이 없고, Tasks 섹션도 불완전합니다. Summary 섹션에 관련 Issue(#141)를 태그하고 구체적인 작업 내용을 작성해주세요. Tasks 섹션도 명확히 작성이 필요합니다.
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.
✅ Passed checks (1 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목이 '[Fix/#141] 내 보드 생성 시 리다이렉트 수정'로 명확하게 주요 변경사항(리다이렉트 수정)을 요약하고 있습니다.
✨ 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 fix/#141

📜 Recent review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 1c4eb99 and e5f07ad.

📒 Files selected for processing (1)
  • src/pages/joinPage/pages/join-complete-page.tsx (2 hunks)
🚧 Files skipped from review as they are similar to previous changes (1)
  • src/pages/joinPage/pages/join-complete-page.tsx

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
Copy link
Copy Markdown

github-actions bot commented Dec 21, 2025

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 387b47c and 5ecbddf.

📒 Files selected for processing (1)
  • src/pages/joinPage/pages/join-complete-page.tsx (1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/pages/joinPage/pages/join-complete-page.tsx (2)
src/stores/useAuthStore.ts (1)
  • useAuthStore (20-75)
src/constants/routes.ts (1)
  • ROUTES (1-33)
🔇 Additional comments (1)
src/pages/joinPage/pages/join-complete-page.tsx (1)

6-6: 인증 스토어 통합이 올바르게 구현되었습니다.

useAuthStore 임포트와 checkAuth 추출이 적절합니다.

Also applies to: 10-10

Copy link
Copy Markdown

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

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

Actionable comments posted: 1

🧹 Nitpick comments (1)
src/pages/joinPage/pages/join-complete-page.tsx (1)

10-10: 코드 스타일 개선 제안

destructuring 시 중괄호 내부에 공백을 추가하는 것이 일반적인 코딩 컨벤션입니다.

🔎 제안하는 수정안
-  const {checkAuth} = useAuthStore();
+  const { checkAuth } = useAuthStore();
📜 Review details

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 5ecbddf and 1c4eb99.

📒 Files selected for processing (1)
  • src/pages/joinPage/pages/join-complete-page.tsx (1 hunks)
🧰 Additional context used
🧬 Code graph analysis (1)
src/pages/joinPage/pages/join-complete-page.tsx (2)
src/stores/useAuthStore.ts (1)
  • useAuthStore (20-75)
src/constants/routes.ts (1)
  • ROUTES (1-33)
🔇 Additional comments (1)
src/pages/joinPage/pages/join-complete-page.tsx (1)

6-6: LGTM!

인증 확인을 위해 필요한 import가 올바르게 추가되었습니다.

@KwonDeaGeun KwonDeaGeun merged commit fe4fba0 into develop Dec 21, 2025
6 checks passed
@KwonDeaGeun KwonDeaGeun deleted the fix/#141 branch December 22, 2025 01:38
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.

[Fix]: 내 보드 생성 완성 시 리다이렉트 수정

1 participant