[OD-169] PostImageSelect, PostUpload 리팩토링 #125
Conversation
| import CommentItem from './CommentItem/index'; | ||
| import MenuButtonList from './MenuButtonList/index'; | ||
|
|
||
| import type { Comment, GetCommentListResponse } from '@apis/post-comment/dto'; | ||
| import type { GetPostLikeListResponse } from '@apis/post-like/dto'; | ||
| import type { ModalProps } from '@components/Modal/dto'; | ||
|
|
||
| import type { LikeCommentBottomSheetProps } from '../dto'; | ||
|
|
||
| import CommentItem from './CommentItem/index'; | ||
| import MenuButtonList from './MenuButtonList/index'; | ||
|
|
There was a problem hiding this comment.
저희가 정한 import문 규칙 상 상대경로가 아래로 가는 것이 맞습니다! 린트 적용해서 수정 부탁드립니당
| import Left from '@assets/arrow/left.svg'; | ||
| import LikeFill from '@assets/default/like-fill.svg'; | ||
| import Like from '@assets/default/like.svg'; | ||
| import Like from '@components/Icons/Like'; |
There was a problem hiding this comment.
이 부분 외에도 린트를 적용해 수정해야 하는 부분들이 보입니다! 전체적인 점검 부탁드립니닷
There was a problem hiding this comment.
함수 정의 사이에 ref 변수가 있습니다 순서 조정 부탁드려용
| const userId = getCurrentUserId(); | ||
| const navigate = useNavigate(); | ||
| const userId = getCurrentUserId(); |
There was a problem hiding this comment.
저는 상단의 user와 postId도 이 단락에 묶이는 것이 더 자연스럽게 읽힐 것 같다는 생각이 드는데 어떠신가용? 생각하신 다른 흐름이 있다면 이대로 유지해도 괜찮습니다!
There was a problem hiding this comment.
또한 userId를 getCurrentUserId에서 받아오면서 이미 userId의 타입이 number인데 아래에 형변환하는 로직이 그대로 남아 있습니다! 수정 부탁드리고, 전체적인 코드에서 userId를 currentUserId로 변경하는 것도 고민해 주시면 좋을 것 같습니다! myId나 currentUserId 등 단순히 userId를 사용하지 않았던 이유가 현재 사용자의 id인지 다른 사용자의 id인지 명확하게 구분하기 위함이었던 데다가, getCurrentUserId 함수에서 받아오는 만큼 currentUserId가 더 적합하다는 생각이 들어서요!
주요 작업 내용
기타 작업 내용
코드 리뷰 포인트
작업 화면