-
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
[FE][Feat] #109 : 손녀 시점에서의 화면 구현 #163
Conversation
onClick: React.MouseEventHandler<HTMLButtonElement>; | ||
} | ||
|
||
// TODO: 이렇게 쓸 거면 범용 버튼으로 빼도...? |
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.
BackButton이라기엔 범용적으로 보이기는 하네요ㅎㅎ 뒤로가기 기능이 여기에 있는거도 아니라 추후에 빼도 좋을거 같습니다
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.
이전 PR인 [FE][Feat} #110 : 뒤로가기 버튼 구현
와 같은 코드가 올라간 것 같아보여요...!!
브랜치는 다른 브랜치로 올리셨는데, 동일한 내용을 올리신 것 같아서,,, 브랜치 별로 해당 기능만 구현해주시고, 그 내용만 PR로 올라갈 수 있게 해주시면 좋을 것 같습니다!
이전 PR에 코멘트 남기고 request changes 해뒀으니 우선 이 PR에는 approve 해둘게요!
크게 손볼게 있어서 다시 PR날리겠습니다! |
📝 PR 개요
🔍 변경 사항
✅ 체크리스트 (Checklist)
🔄 관련 이슈 (Linked Issues)
close #109
📷 스크린샷 및 동영상 (선택 사항)
🧪 테스트 방법
📚 참고 자료 (선택 사항)