[ Feat ] router 연결 #14
Conversation
There was a problem hiding this comment.
수고하셨습니다 🩵
한가지 제안 드리자면,
현재 페이지 컴포넌트 (각 페이지의 최상위 컴포넌트) 명이 path (camelCase -> PascalCase) 로 되어있는데, 혹시 이걸 path+Page 로 바꾸는건 어떨까요?
ex) <HomePage />, <SeniorOnboardingPage />, <SeniorProfilePage /> ...
페이지 내부에서 사용하는 컴포넌트가 아닌, 페이지 최상위 컴포넌트라는걸 네이밍을 통해 직관적으로 파악 가능하게 하면 좋을 것 같아 제안드립니다!
src/Router.tsx
Outdated
| { | ||
| path: 'seniorProfile', | ||
| element: <SeniorProfile />, | ||
| }, | ||
| { | ||
| path: 'juniorProfile', | ||
| element: <JuniorPromise />, | ||
| }, |
There was a problem hiding this comment.
p2) 요기 juniorProfile -> <JuniorPromise /> 으로 되어있는데 seniorProfile 과의 일관성을 위해 여기서도 컴포넌트명까지 <JuniorProfile로 /> 다 통일하는거 어떠신가요??
There was a problem hiding this comment.
p2) 아니면 Path를 juniorPromise로 하는 건 어떤가요 ? 후배약속잡는뷰이니깐 !
There was a problem hiding this comment.
p5) 선배프로필 보는 부분, 주니어가 약속잡는거 직관적이고 좋은 거 같아욤!
src/Router.tsx
Outdated
| { | ||
| path: 'seniorOnboarding', | ||
| element: <SeniorOnboarding />, | ||
| }, | ||
| { | ||
| path: 'juniorOnboarding', | ||
| element: <JuniorOnboarding />, | ||
| }, |
There was a problem hiding this comment.
Onboarding이 두 플로우가 동일한 path를 사용하게 될 수도 있을 것 같은데, 일단 이렇게 해두고 제가 작업하면서 path 통합하는게 낫겠다! 싶으면 그때 수정하겠습니다 :)
src/Router.tsx
Outdated
| { | ||
| path: 'seniorProfile', | ||
| element: <SeniorProfile />, | ||
| }, | ||
| { | ||
| path: 'juniorProfile', | ||
| element: <JuniorPromise />, | ||
| }, |
There was a problem hiding this comment.
p2) 아니면 Path를 juniorPromise로 하는 건 어떤가요 ? 후배약속잡는뷰이니깐 !
| element: <JuniorPromisePage />, | ||
| }, | ||
| ], | ||
| }, |
There was a problem hiding this comment.
p2) 2차 스프린트로 넘어간 부분이니까 없어도 괜찮을 거 같아요!!
#️⃣ Related Issue
Closes #13
✅ Done Task
☀️ New-insight
React Router v6에서 도입된 새로운 라우팅 설정 방법
복잡한 라우팅 구조를 보다 직관적으로 설정
주요 구성 요소
createBrowserRouterRouterProviderRouterProvider의router속성에 전달하여 사용필요한 패키지 설치
→ 이미 설치되어있는거
package.json에서 확인하고 패스💎 PR Point
📸 Screenshot