Skip to content

Comments

Fix(client): change onboarding path url small letter#238

Merged
constantly-dev merged 1 commit intodevelopfrom
fix/#237-onboarding-path-small-letter-change
Jan 7, 2026
Merged

Fix(client): change onboarding path url small letter#238
constantly-dev merged 1 commit intodevelopfrom
fix/#237-onboarding-path-small-letter-change

Conversation

@constantly-dev
Copy link
Member

@constantly-dev constantly-dev commented Jan 7, 2026

📌 Related Issues

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

📄 Tasks

온보딩 path url 소문자로 변경

⭐ PR Point (To Reviewer)

  • startsWith 등 대소문자 구분하는 곳에서 에러가 터져 이를 해결하기 위해 모두 소문자로 통일하도록 변경
  • (onBoarding -> onboarding)

Summary by CodeRabbit

리팩토링

  • 온보딩 및 콜백 관련 URL 경로를 소문자 형식으로 표준화했습니다. '/onBoarding' 형식이 '/onboarding'으로 변경되었으며, '/onBoarding/callback'이 '/onboarding/callback'으로 변경되었습니다.

버그 수정

  • 인증 프로세스 리다이렉트 경로를 수정했습니다.

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

@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 5:27pm
pinback-client-landing Ready Ready Preview, Comment Jan 7, 2026 5:27pm

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

coderabbitai bot commented Jan 7, 2026

Walkthrough

onboarding 관련 route 설정의 대소문자를 통일하는 변경으로, 4개 파일에서 onBoardingonboarding, onBoardingCallbackonboardingCallback으로 수정하고 경로도 /onBoarding/onboarding으로 일관성 있게 변경합니다.

Changes

Cohort / File(s) 변경 사항
Route Config 키 이름 변경
apps/client/src/routes/routesConfig.ts
route config 객체의 키를 camelCase 소문자로 변경: onBoardingonboarding, onBoardingCallbackonboardingCallback; 해당 경로도 업데이트: /onBoarding/onboarding, /onBoarding/callback/onboarding/callback
Route 참조 업데이트
apps/client/src/routes/router.tsx, apps/client/src/layout/Layout.tsx
ROUTES_CONFIG 참조 키를 route config 변경에 맞게 업데이트 (onBoarding → onboarding 등)
axios Interceptor 리다이렉트 경로 수정
apps/client/src/shared/apis/setting/axiosInstance.ts
response interceptor의 리다이렉트 경로를 /onBoarding?step=SOCIAL_LOGIN/onboarding?step=SOCIAL_LOGIN으로 수정

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Possibly related PRs

  • #237: onboarding 관련 경로를 소문자로 변경하는 동일한 수정 작업
  • #233: Layout.tsx의 isSidebarHidden 계산 로직 수정으로 같은 파일에 영향
  • #226: axios interceptor의 onboarding 리다이렉트 처리 관련 수정

Suggested labels

fix

Suggested reviewers

  • jllee000
  • jjangminii

Poem

🐰 대문자에서 소문자로,
onBoarding이 onboarding 되었네요!
경로도 일관되게 정리하고,
가지런한 코드의 미학이 피어나네~ ✨

🚥 Pre-merge checks | ✅ 4 | ❌ 1
❌ Failed checks (1 warning)
Check name Status Explanation Resolution
Linked Issues check ⚠️ Warning 변경 사항이 #237의 요구사항(onboarding 경로 소문자 정규화)을 충족하지만, #25(Progress 컴포넌트)는 이 PR과 무관합니다. #25는 Progress 컴포넌트 구현으로 현재 PR과 관련이 없습니다. 관련된 이슈는 #237만 참조하도록 수정하십시오.
✅ Passed checks (4 passed)
Check name Status Explanation
Title check ✅ Passed PR 제목은 주요 변경 사항인 onboarding 경로를 소문자로 변경하는 것을 명확하게 설명합니다.
Description check ✅ Passed PR 설명이 관련 이슈(#237), 작업 내용, 리뷰어를 위한 주요 포인트를 포함하여 필수 템플릿 구조를 따릅니다.
Out of Scope Changes check ✅ Passed 모든 변경 사항이 #237의 onboarding 경로 소문자 변경 목표에 부합합니다. 범위를 벗어난 변경 없음.
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 59b10a1 and 0efd70e.

📒 Files selected for processing (4)
  • apps/client/src/layout/Layout.tsx
  • apps/client/src/routes/router.tsx
  • apps/client/src/routes/routesConfig.ts
  • apps/client/src/shared/apis/setting/axiosInstance.ts
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings
Learnt from: constantly-dev
Repo: Pinback-Team/pinback-client PR: 2
File: pnpm-workspace.yaml:3-3
Timestamp: 2025-08-18T13:48:59.065Z
Learning: constantly-dev는 docs 디렉터리를 컨벤션 문서 추가용으로 사용할 예정이라고 명시했습니다.
Learnt from: constantly-dev
Repo: Pinback-Team/pinback-client PR: 5
File: apps/extension/src/index.css:1-1
Timestamp: 2025-08-19T17:18:57.678Z
Learning: constantly-dev는 디자인 시스템 설정 PR 머지 후 `import 'pinback/tailwind-config/shared-styles.css';`를 `app.css`나 `index.css`에서 사용하여 공유 스타일을 관리할 계획입니다.
🧬 Code graph analysis (2)
apps/client/src/routes/router.tsx (1)
apps/client/src/routes/routesConfig.ts (1)
  • ROUTES_CONFIG (1-38)
apps/client/src/layout/Layout.tsx (1)
apps/client/src/routes/routesConfig.ts (1)
  • ROUTES_CONFIG (1-38)
⏰ 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/shared/apis/setting/axiosInstance.ts (1)

50-50: 변경사항 확인 완료!

onboarding 경로가 소문자로 올바르게 업데이트되었습니다. 새로운 라우트 설정과 일관성을 유지합니다.

apps/client/src/routes/router.tsx (1)

36-41: 라우트 설정 업데이트 확인 완료!

라우트 키가 소문자로 정확하게 업데이트되었으며, routesConfig.ts의 변경사항과 일치합니다.

apps/client/src/layout/Layout.tsx (1)

12-16: 케이스 민감 체크 이슈 해결 확인!

startsWith() 메서드는 대소문자를 구분하므로, 소문자로 정규화된 경로를 사용하는 것이 올바릅니다. 이는 PR #237에서 언급된 케이스 민감 체크 오류를 직접적으로 해결합니다.

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

18-25: 라우트 설정 정규화 완료!

키 이름과 경로가 소문자로 일관되게 업데이트되었습니다. 코드베이스 전체에서 이전 onBoarding, /onBoarding, onBoardingCallback 패턴에 대한 하드코딩된 참조가 남아있지 않음을 확인했습니다.


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

github-actions bot commented Jan 7, 2026

✅ Storybook chromatic 배포 확인:
🐿️ storybook

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.

굿쿳~

@constantly-dev constantly-dev merged commit d2dfc00 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] onboarding 관련 path 소문자로 변경

2 participants