-
Notifications
You must be signed in to change notification settings - Fork 8
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
✨ 프로필 이미지 변경 기능 구현 #119
✨ 프로필 이미지 변경 기능 구현 #119
Conversation
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.
고생하셨습니다! 혹시 사진 업로드 버튼 UI는 추후에 따로 업데이트를 할 예정인가요?
useEffect(() => () => { | ||
if (userSocketRef.current) { | ||
userSocketRef.current.disconnect(); | ||
} | ||
}, []); | ||
|
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.
deps에 user가 있어서 따로 빼둔 걸까여요?
@@ -34,7 +34,7 @@ const LargeProfileImageBox = styled.img` | |||
border-radius: 30%; | |||
|
|||
&:hover{ | |||
cursor: pointer; | |||
cursor: ${(props: {isMine:boolean}) => (props.isMine && 'pointer')}; |
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.
오 이런 디테일한 부분까지도 신경 써주셔서 좋네요 ㅎㅎ
네, 주로 인풋부분을 감추로 라벨을 달아서 스타일링 하는 방식으로 구현하는 것 같은데 목록에 넣어두고 있습니다..! |
개요
✨ 프로필 이미지 변경 기능 구현
작업사항
jwt-util
에서 알고리즘과 만료기한을 리뷰받은 내용을 참고해 환경변수로 설정해주었습니다.변경후
2021-11-23.9.18.04.mov