Skip to content

Conversation

@xxinzzi
Copy link
Collaborator

@xxinzzi xxinzzi commented Nov 30, 2024

주요 작업 내용

  • 선택한 댓글이 아닌 맨 밑 댓글이 삭제되는 오류 해결했습니다.
  • CommentItem 내부에 있던 로직들 LikeCommentBottomSheet으로 이동했습니다.
    (댓글 api 및 모달 띄우는 로직이 CommentItem 밖에 있는 게 더 적절한 듯 하여..)
  • api 연결 실패 시 띄우는 모달 구현했습니다.
  • 댓글 신고 api는 없어 추후에 생긴다면 연결해야합니다.

기타 작업 내용

  • MenuButtonList UI 위치 올바르게 수정하였습니다.
  • apis/post-comment/dto 에서 CommentUser 내부에 오타가 있어 userId -> id 로 수정했습니다.
  • ClothingInfoItem의 props 중 hasRightMargin 삭제했습니다.
    (ClothingInfoItem을 매핑하는 컨테이너에서 gap을 주는 것으로 변경)
  • PostBase UI 수정하였습니다.
    (content 더보기를 눌렀을 때 사진이 아래로 밀리는게 안좋은 것 같아서.. content를 맨 밑으로 내렸습니다.)

코드 리뷰 포인트

  • 없음

작업 화면

스크린샷 2024-11-30 오후 9 16 01

내 댓글 메뉴 버튼 클릭시

스크린샷 2024-11-30 오후 9 16 09

다른 유저 댓글 메뉴 버튼 클릭 시

스크린샷 2024-11-30 오후 9 16 23

@xxinzzi xxinzzi added fix Fix bugs that don't work feat New feature labels Nov 30, 2024
@xxinzzi xxinzzi self-assigned this Nov 30, 2024
@xxinzzi xxinzzi changed the title [OD-128] 댓글 삭제 오류 해결 [OD-128] 게시물 댓글 오류 해결 Nov 30, 2024
@xxinzzi xxinzzi changed the title [OD-128] 게시물 댓글 오류 해결 [OD-128] PostBase 컴포넌트 오류 해결 Nov 30, 2024
@lalaurrel
Copy link
Collaborator

고생했다아 ~~~

Comment on lines 197 to 198
// 유저 차단 api
const sendBlock = async () => {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

api호출함수에 맞춰서 postUserBlock으로 수정해주세용

Comment on lines 232 to 259
? [
{
text: '삭제',
action: () => {
setIsCommentDeleteConfirmationModalOpen(true);
setIsMenuVisible(false);
},
icon: Delete,
color: 'red',
},
]
: [
{
text: '신고하기',
action: () => {
setIsCommentReportModalOpen(true);
},
icon: Report,
},
{
text: '차단하기',
action: () => {
setIsBlockConfirmationModalOpen(true);
setIsMenuVisible(false);
},
icon: Block,
},
]),
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

내 댓글 메뉴와 다른 사람 댓글 메뉴에 대한 객체 배열을 별도의 변수로 분리해서 작성하면 가독성이 좋아질 것 같아용 한번 고려해 보시고 더 괜찮은 방식으로 사용하시면 될 것 같습니다!!

Comment on lines -5 to +9
import { UserBlockAtom } from '../../../recoil/Home/BlockBottomSheetAtom';
import {
IsCommentDeleteConfirmationModalOpenAtom,
IsCommentReportModalOpenAtom,
selectedCommentAtom,
} from '../../../recoil/Post/PostCommentAtom';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/recoil/Home/BlockBottomSheetAtom 더 이상 사용 안 하시면 삭제해 주세요! Home 폴더 자체를 삭제하면 될 것 같습니다~~

@gustn99 gustn99 merged commit 083a14f into dev Dec 1, 2024
1 check passed
@gustn99 gustn99 deleted the feat/OD-128 branch December 1, 2024 09:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feat New feature fix Fix bugs that don't work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants