-
Notifications
You must be signed in to change notification settings - Fork 0
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/i39] 로그아웃 버튼 생성 #54
Conversation
Conflicts: src/components/common/AvatarModal.tsx
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
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.
굿 👍 개인적으로 선호하는 코드 스타일에 대한 코멘트 하나 남겨두었습니다! 작업하시느라 수고하셨어용 😄
} | ||
|
||
function AvatarModal({ open, onClose }: AvatarModalProps) { | ||
const [logoutOpen, setLogoutOpen] = useState(false); |
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.
useState 바로 뒤에 <현재 state의 타입> 방식으로 타입을 명시해두면 혹시라도 나중에 코드 다시 볼일이 생길 때 코드 파악하기 편해서 추천드립니당 👍
코드 예시)
const [logoutOpen, setLogoutOpen] = useState<boolean>(false);
🛠️ 변경사항
☝️ 유의사항
❗체크리스트