Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[setting] Route 및 절대 경로 설정 #6

Open
wants to merge 5 commits into
base: develop
Choose a base branch
from

Conversation

heesunee
Copy link
Collaborator

@heesunee heesunee commented Jan 7, 2025

📌 Related Issues

📄 Tasks

  • 절대 경로 설정
  • Route 설정

✅ PR Point (To Reviewer)

  • 추후에 페이지 별로 폴더 파지면 '/' 에 대신 home?이 들어가면 될 것 같아요
  • 틀린 것, 평소 자기가 작성하던 방식이랑 다른 점 있다면 남겨주세요
  • 노트북이 항상 엔터치면 마지막 글자가 한번 더 써져서 커밋 메시지가 절대경로 설정정 이딴식이네요. 참고 바람.

📷 Screenshot

(테스트용 파일들을 삭제 후 커밋함)

image

image

🔔 ETC

  • tsconfig.json과 tsconfig.app.json의 차이점!
  • tsconfig.json이 공통설정을 하는 곳은 맞지만, vite 설정에서 tsconfig.json에 tsconfig.app.json과 tsconfig.node.json을 참조한다고 작성되어있기 때문에 tsconfig.app.json에 작성해야 절대경로 설정이 적용된다.
  • https://www.notion.so/Route-adda7b7764404dffb1a8b55e10bf0825?pvs=4 나의 초기세팅 여정.. 안까먹을려고 정리~

@heesunee heesunee added ⚙️ Setting 개발 환경 세팅 희선 자중해 labels Jan 7, 2025
@heesunee heesunee self-assigned this Jan 7, 2025
Copy link
Collaborator

@rtttr1 rtttr1 left a comment

Choose a reason for hiding this comment

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

LGTM!!

@constantly-dev
Copy link
Collaborator

constantly-dev commented Jan 7, 2025

수고하셨습니다! 깔끔하게 잘 해주셨네요.
해당 코드에서 약간의 생각을 더해보자면 route path를 문자열로 직접 사용하는 것이 아니라 상수화를 해서 한 파일에서 관리하면 유지 보수와 가독성이 좋아질 수 있을 것 같습니다!

예시 코드)

const authPages = {
  LOGIN: '/',
  SIGN_UP: '/signup',
};

const myPagePages = {
  MY_PAGE: '/users/me',
  MY_PAGE_INFO: '/users/me/info',
};

이렇게 상수화를 해서 아래와 같이 사용하는 방법도 있을 것 같아서 의견 내봅니다!

const authRoutes: RouteType[] = [
  {
    path: routePath.LOGIN,
    element: <Login />,
  },
  {
    path: routePath.SIGN_UP,
    element: <SignUp />,
  },
];

@KIMGEONHWI
Copy link
Member

KIMGEONHWI commented Jan 7, 2025

수고하셨습니다! 깔끔하게 잘 해주셨네요. 해당 코드에서 약간의 생각을 더해보자면 route path를 문자열로 직접 사용하는 것이 아니라 상수화를 해서 한 파일에서 관리하면 유지 보수와 가독성이 좋아질 수 있을 것 같습니다!

예시 코드)

const authPages = {
  LOGIN: '/',
  SIGN_UP: '/signup',
};

const myPagePages = {
  MY_PAGE: '/users/me',
  MY_PAGE_INFO: '/users/me/info',
};

이렇게 상수화를 해서 아래와 같이 사용하는 방법도 있을 것 같아서 의견 내봅니다!

const authRoutes: RouteType[] = [
  {
    path: routePath.LOGIN,
    element: <Login />,
  },
  {
    path: routePath.SIGN_UP,
    element: <SignUp />,
  },
];

저도 해당 의견에 동의합니다.

src/router/routesConfig.ts

// routesConfig.ts
export const ROUTES_CONFIG = {
	home: {
		title: 'Home',
		path: '/home',
	},
	onboarding: {
		title: 'Onboarding',
		path: '/onboarding',
	},
	timer: {
		title: 'Timer',
		path: '/timer',
	},
};

위와 같이 routesConfig.ts 파일 생성해서 중앙 집중식 관리하면 좋을 것 같아요!

Copy link
Collaborator

@hansoojeongsj hansoojeongsj left a comment

Choose a reason for hiding this comment

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

최고 !! 고생하셨습니당
저도 진혁 님 의견처럼 상수 사용했을 때 더 편리하고 좋았던 것 같아요 :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⚙️ Setting 개발 환경 세팅 희선 자중해
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[setting] Route 및 절대 경로 설정
5 participants