-
Notifications
You must be signed in to change notification settings - Fork 2
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
[소그룹스터디] 경로를 static으로 변경 #418
Conversation
@@ -43,7 +44,7 @@ function HotPost() { | |||
hotArticleList.map((hotPost: HotPostResponse, index: number) => ( | |||
<Link | |||
className={styles.hotpost__content} | |||
to={`/board/notice/${hotPost.id}`} | |||
to={`${ROUTES.BOARD_NOTICE_DETAIL.replace(':id', String(hotPost.id))}`} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
route를 함수로 바꿔쓰는건 어떻까용?
인터페이스가 꽤나 불편해보여용
Routes.BoardNoticeDetail({id: hotPost.id})
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
구조 개편 완료했습니다. caeeac7
src/static/routes.ts
Outdated
AuthModifyInfo: '/auth/modifyinfo', | ||
Review: { | ||
path: 'review/:id', | ||
general: (id: string) => `/review/${id}`, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
2개 이상의 path를 쓰는 경우를 대비해서 객체로 받는건 어떨까욤
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
수고하셨습니다.
원빈이가 지적해준 내용들 다 처리가 된건가요??
확장성 있게 변경했습니다~ |
What is this PR? 🔍
Changes 📝
ScreenShot 📷
Test CheckList ✅
Precaution
✔️ Please check if the PR fulfills these requirements
develop
branch unconditionally?main
?yarn lint