Skip to content

Conversation

@starboxxxx
Copy link
Collaborator

@starboxxxx starboxxxx commented Jan 31, 2025

📄 요약(Summary)

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 댓글 첨부파일 삭제에서 기존에는 삭제된 첨부파일의 ID를 리스트로 받은 후 이를 이용하여 repository에서 해당 첨부파일을 찾아서 softdelete를 진행하였습니다.
  • 그러나 댓글 첨부파일은 댓글과 1:1 관계이기 때문에 그냥 CommentId만으로도 attachmentrepository에서 찾을 수 있었고 이를 고려하여 리팩토링 작업을 진행하였습니다.

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요
ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

PR Log

PR 작업하면서 고민했던 내용, 해결한 내용, 고민 중인 내용 등

새롭게 배운 것

고민 중인 사항

첨부 자료

스크린샷 2025-01-31 오후 6 55 24

@joowojr @parkjaehak 여기 로직 어떤 부분 때문에 필요한지 궁금합니다!
댓글 첨부파일 logic에도 저게 필요한지 궁금해서 문의드립니다

Requirements for Reviewer

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

✅ 체크리스트(Checklist)

  • [ ✅ ] PR 양식에 맞게 작성했습니다
  • [ ✅ ] 모든 테스트가 통과했습니다
  • [ ✅ ] 프로그램이 정상적으로 작동합니다
  • [ ✅ ] 적절한 PR 라벨을 설정했습니다
  • [ ✅ ] 불필요한 코드를 제거했습니다

🚪 이슈 번호(Issue numbers)

Closes #{이슈 번호}

@starboxxxx starboxxxx added 🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다. MID 우선순위 중 labels Jan 31, 2025
@starboxxxx starboxxxx self-assigned this Jan 31, 2025
@starboxxxx starboxxxx requested a review from joowojr January 31, 2025 10:01
@starboxxxx starboxxxx merged commit 49411fc into develop Feb 1, 2025
1 check passed
@joowojr joowojr deleted the CLAP-185 branch February 3, 2025 06:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🧽 cleanup 코드를 더 깔끔하게 만들기만 하고, 코드 작동 방식이나 출력에 대한 부분을 변경하지 않습니다. MID 우선순위 중 ♻️ refactor 코드가 내부적으로 작동하는 방식을 변경합니다. cleanup과는 다릅니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants