Skip to content

Conversation

@jjangminii
Copy link
Collaborator

@jjangminii jjangminii commented Jan 7, 2026

📌 Related Issues

관련된 Issue를 태그해주세요. (e.g. - close #25)

  • close #

📄 Tasks

⭐ PR Point (To Reviewer)

📷 Screenshot

Summary by CodeRabbit

릴리스 노트

  • 버그 수정
    • 온보딩 네비게이션 경로 표준화로 라우팅 일관성 개선
    • 콜백 및 리다이렉트 경로 정규화로 사용자 흐름 안정성 향상

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

@jjangminii jjangminii linked an issue Jan 7, 2026 that may be closed by this pull request
@vercel
Copy link

vercel bot commented Jan 7, 2026

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

Project Deployment Review Updated (UTC)
pinback-client-client Ready Ready Preview, Comment Jan 7, 2026 4:08pm
pinback-client-landing Ready Ready Preview, Comment Jan 7, 2026 4:08pm

@github-actions github-actions bot added the fix 버그 수정하라 러브버그 label Jan 7, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 7, 2026

Walkthrough

온보딩 라우트 경로의 대소문자 표기를 일관되게 수정하는 PR입니다. '/onboarding'을 '/onBoarding'으로 변경하여 camelCase 컨벤션을 적용했으며, 네 개 파일의 네비게이션 및 리다이렉트 경로에서 총 7개 지점을 업데이트했습니다.

Changes

Cohort / File(s) 변경 사항
onBoarding 페이지 라우팅
apps/client/src/pages/onBoarding/GoogleCallback.tsx, apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx
네비게이션 경로를 '/onboarding?step=...' → '/onBoarding?step=...'으로 변경 (각 파일 3개 지점, 총 6개 지점)
라우트 설정
apps/client/src/routes/routesConfig.ts
ROUTES_CONFIG.onBoardingCallback.path에서 '/onboarding/callback' → '/onBoarding/callback'으로 변경
API 에러 핸들링
apps/client/src/shared/apis/setting/axiosInstance.ts
401/403 에러 처리 시 리다이렉트 경로를 '/onboarding?step=SOCIAL_LOGIN' → '/onBoarding?step=SOCIAL_LOGIN'으로 변경

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 분

변경 내용이 동일한 패턴의 반복적 경로 문자열 수정이므로, 각 파일의 변경이 일관되고 예측 가능합니다.

Possibly related PRs

Suggested labels

fix

Suggested reviewers

  • jllee000
  • constantly-dev

Poem

🐰 경로가 헷갈렸더니
camelCase로 고쳐두니
온보딩이 반짝반짝✨
네비게이션도 쌩쌩!
작은 수정, 큰 정리~

🚥 Pre-merge checks | ✅ 2 | ❌ 3
❌ Failed checks (3 warnings)
Check name Status Explanation Resolution
Description check ⚠️ Warning PR 설명에 관련 이슈 번호가 명시되지 않았고(close # 만 있음), 작업 내용, 리뷰어 포인트, 스크린샷 등 필수 섹션이 모두 비어있습니다. 관련 이슈 번호를 명시하고, 작업 내용(URL 경로 대소문자 정규화 이유)을 요약하여 작성해주세요.
Linked Issues check ⚠️ Warning PR의 URL 경로 대소문자 정규화 변경사항은 링크된 이슈 #25(Progress 컴포넌트 구현)와 전혀 관련이 없습니다. 올바른 관련 이슈를 PR 설명에 명시하거나, 이슈 #25와 관계없는 변경사항임을 확인하고 적절한 이슈로 수정해주세요.
Out of Scope Changes check ⚠️ Warning PR 변경사항(URL 경로 대소문자 정규화)은 링크된 이슈 #25(Progress 컴포넌트 구현)의 범위를 벗어나 있습니다. PR 설명에 올바른 관련 이슈 번호를 명시하거나, 이 변경사항이 다른 이슈와 관련되어 있는지 확인하세요.
✅ Passed checks (2 passed)
Check name Status Explanation
Title check ✅ Passed 제목은 무한 로딩 오류 수정이라는 주요 변경사항을 명확하게 요약하고 있으며, 실제 코드 변경(URL 경로 대소문자 정규화)과 관련이 있습니다.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
  • 📝 Generate docstrings

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 9629a80 and fd5af0b.

📒 Files selected for processing (4)
  • apps/client/src/pages/onBoarding/GoogleCallback.tsx
  • apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx
  • apps/client/src/routes/routesConfig.ts
  • apps/client/src/shared/apis/setting/axiosInstance.ts
🧰 Additional context used
🧬 Code graph analysis (1)
apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx (1)
apps/client/src/pages/onBoarding/constants/onboardingSteps.ts (1)
  • Step (1-9)
⏰ 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: storybook
🔇 Additional comments (4)
apps/client/src/pages/onBoarding/GoogleCallback.tsx (1)

15-15: 경로 변경이 일관되게 적용되었습니다.

세 곳의 네비게이션 지점(코드 누락, 신규 사용자, 에러 처리)에서 모두 /onBoarding 경로로 일관되게 업데이트되었습니다.

단, axiosInstance.ts에서 언급한 URL 명명 규칙 관련 우려사항이 여기에도 동일하게 적용됩니다.

Also applies to: 39-39, 54-54

apps/client/src/pages/onBoarding/components/funnel/MainCard.tsx (1)

167-167: 퍼널 네비게이션 경로가 일관되게 업데이트되었습니다.

온보딩 퍼널의 세 가지 네비게이션 지점(Mac 스텝 스킵, 다음 스텝, 이전 스텝)에서 모두 /onBoarding 경로로 일관되게 변경되었습니다. 템플릿 리터럴을 사용한 동적 스텝 파라미터 처리도 올바르게 유지되고 있습니다.

Also applies to: 187-187, 196-196

apps/client/src/routes/routesConfig.ts (1)

24-24: 라우트 설정이 올바르게 적용되었습니다.

경로가 /onBoarding/callback으로 변경된 것은 부모 경로 /onBoarding과 일치시키기 위한 올바른 수정입니다. router.tsx에서 ROUTES_CONFIG.onBoardingCallback.path를 통해 올바르게 참조되고 있으며, GoogleCallback.tsx의 모든 네비게이션도 /onBoarding으로 통일되어 있습니다. 이 경로 불일치가 무한 로딩 오류의 원인이었을 것으로 보입니다.

apps/client/src/shared/apis/setting/axiosInstance.ts (1)

50-50: 라우트 경로가 올바르게 정의되어 있습니다.

/onBoarding 경로는 apps/client/src/routes/routesConfig.ts 라인 20에서 정의되어 있으며, 리다이렉트 경로가 일치합니다. 전체 코드베이스에서 디렉토리 구조(pages/onBoarding/), 라우트 설정, 리다이렉트 경로 모두 camelCase 규칙을 일관되게 사용하고 있습니다.

URL에 camelCase를 사용하는 것은 표준 관례(kebab-case)와 다르지만, 이는 코드베이스 전체의 의도된 스타일 선택이므로 이 변경사항 자체에는 문제가 없습니다.


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.

Copy link
Collaborator

@jllee000 jllee000 left a comment

Choose a reason for hiding this comment

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

한밤중의 이슈,, 그래도 빠르게 해결돼서 다행임다!! 굿굿

@github-actions
Copy link

github-actions bot commented Jan 7, 2026

✅ Storybook chromatic 배포 확인:
🐿️ storybook

@jjangminii jjangminii merged commit c689234 into develop Jan 7, 2026
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix 버그 수정하라 러브버그

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fix] 무한로딩 에러

3 participants