Skip to content
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: 댓글 파일 첨부 구현 #279

Merged
merged 13 commits into from
Dec 11, 2024
Merged

feat: 댓글 파일 첨부 구현 #279

merged 13 commits into from
Dec 11, 2024

Conversation

imdaxsz
Copy link
Contributor

@imdaxsz imdaxsz commented Dec 11, 2024

📝 개요

댓글 파일 첨부 구현

image

댓글(파일) 수정

image

이미 생성된 댓글에 첨부된 파일을 삭제하려면 댓글 자체를 삭제해야 합니다.


🚀 변경사항

  • PostNavigation과 댓글 영역 사이에 간격을 좀 더 크게 했습니다.
  • getFileName 예외 처리 추가
  • getFileName 예외 처리 추가 이후에 useSubClassFileHandler 테스트가 실패하는 문제가 있었습니다. fileData.filePresignedUrl이 undefined인 경우에도 함수가 호출되는 것 같아 isPdfFile 값을 수정했습니다. 0c2ee90

🔗 관련 이슈

#272


➕ 기타

@imdaxsz imdaxsz added feature 새 기능 추가 fix 텍스트, 오타 수정 등 labels Dec 11, 2024
@imdaxsz imdaxsz requested a review from PHJoon December 11, 2024 04:41
@imdaxsz imdaxsz self-assigned this Dec 11, 2024
@imdaxsz imdaxsz linked an issue Dec 11, 2024 that may be closed by this pull request
4 tasks
Copy link

cloudflare-workers-and-pages bot commented Dec 11, 2024

Deploying abcdedu-frontend-dev with  Cloudflare Pages  Cloudflare Pages

Latest commit: 6c4c9ae
Status: ✅  Deploy successful!
Preview URL: https://78103be8.abcdedu-frontend.pages.dev
Branch Preview URL: https://feat-comment-file-cmh.abcdedu-frontend.pages.dev

View logs

Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

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

고생하셨습니다!
댓글 수정할 때 첨부파일은 안건들고 내용만 수정한 후에 완료하면 첨부파일 사라지는 것 같은데 확인해주실 수 있을까요?
(아무것도 안건들고 완료버튼 눌러도 위와 같이 되는 것 같습니다.)

handleClose,
fileName,
onSubmit,
} = useCommmentForm({ postId, comment, toggleEditMode });
Copy link
Contributor

Choose a reason for hiding this comment

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

useCommmentForm 오타 난 것 같아요

Copy link
Contributor

@PHJoon PHJoon left a comment

Choose a reason for hiding this comment

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

확인했습니다! 🚀

@imdaxsz imdaxsz merged commit 9035af7 into develop Dec 11, 2024
2 checks passed
@imdaxsz imdaxsz deleted the feat/comment-file/cmh branch December 11, 2024 11:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature 새 기능 추가 fix 텍스트, 오타 수정 등
Projects
None yet
Development

Successfully merging this pull request may close these issues.

댓글 파일 첨부 기능 구현
2 participants