-
Notifications
You must be signed in to change notification settings - Fork 3
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
[Feat/#91] 캐러셀 라우팅 및 자잘한 변경 #101
Conversation
커밋을 잘 나누고 있어서 흐름 파악하기 아주 좋네요. 다만 스타일만 변경한 부분은 fix말고 style 커밋메세지 사용해주세요! |
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.
lgtm
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.
lgtm
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.
헤더 네이밍만 수정하고 머지해주세요
export const LogOutHeader = () => { | ||
const navigate = useNavigate(); | ||
const handleLogIn = () => { | ||
navigate(`/login`); | ||
}; |
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.
[P3]
헤더네이밍이 unAuthorizationHeader 면 좋을 것 같아요
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.
넵 수정하겠습니다!
✨ 해당 이슈 번호 ✨
closed #91
todo
📌 내가 알게 된 부분
📌 질문할 부분
📌스크린샷(선택)