[FE][Feat} #110 : 뒤로가기 버튼 구현 #162
Closed
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
frontend 폴더에 import { App } from './App2'; 가 언제 들어갔죠... 코드리뷰가 부족했었나보네요. 더 열심히 살펴보겠습니다 |
happyhyep
requested changes
Nov 14, 2024
| @@ -2,7 +2,7 @@ import { StrictMode } from 'react'; | |||
| import { createRoot } from 'react-dom/client'; | |||
| import { BrowserRouter } from 'react-router-dom'; | |||
| import './index.css'; | |||
| import { App } from './App2'; | |||
| import { App } from './App'; | |||
| // 우선은 폰트 다 포함시켰는데, 나중에 사용할 것들만 따로 뺴자. | |||
There was a problem hiding this comment.
이것도 만약 추후에 작업하시려고 주석처리 해둔 거라면, TODO:로 빼주시면 좋을 것 같아요!
| // import { MdInfo } from 'react-icons/md'; | ||
|
|
||
| export const GuestView = () => { | ||
| // TODO: 헤더 테스트 로직 추후 범용 Header로 제작시 제거 예정 |
There was a problem hiding this comment.
여기는 다 주석이라... 이렇게 다 주석과 TODO로 해두는 것보다는,,, //TODO GuestView 구현 필요 정도의 주석 한 줄만 있는게 더 좋을 것 같아 보여요! 물론 금방 추가하시겠지만 너무 주석밖에 없는 것 같아서... dev 브랜치에 머지되기에는 어려움이 있어보입니다..!!
| @@ -1 +1,23 @@ | |||
| export const GuestView = () => <>Hello</>; | |||
| import { Header } from '@/component/Layout/header/Header.tsx'; | |||
| // import { MdInfo } from 'react-icons/md'; | |||
There was a problem hiding this comment.
이것도 나중에 쓰실거라면 주석 말고 그냥 제거해두는게 더 좋을 것 같아요!
| navigate(-1); | ||
| }; | ||
|
|
||
| // TODO: h1으로 걸거나, h2로 거는건 어케걸지 생각좀 하기 |
There was a problem hiding this comment.
그래도 공식적으로 올라가는 코드이니까, 조금 더 주석이 official 느낌 나게 작성되면 좋을 것 같습니다..!! 현재는 너무 메모장 느낌이라... 팀원 모두가 알아볼 수 있는 TODO였으면 좋겠어요!
|
크게 손볼게 있어서 다시 PR날리겠습니다. |
effozen
added
작업 중지
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📝 PR 개요
🔍 변경 사항
✅ 체크리스트 (Checklist)
🔄 관련 이슈 (Linked Issues)
📷 스크린샷 및 동영상 (선택 사항)
🧪 테스트 방법
📚 참고 자료 (선택 사항)