Skip to content

Conversation

@starboxxxx
Copy link
Collaborator

@starboxxxx starboxxxx commented Jan 30, 2025

📄 요약(Summary)

댓글 수정 및 삭제 API 구현

✍🏼 상세(More)

PR Desciption

변경 사항 설명

  • 댓글도 soft Delete가 필요하다고 판단하여 댓글 entity에 isDeleted 컬럼을 추가하였습니다.

  • 첨부파일 댓글의 경우에는 별도의 수정 API가 필요없다고 tony와 얘기되어 댓글 수정은 일반 댓글만 가능하도록 구현해놓았습니다!

  • 삭제 api를 하나로 구성하였습니다.

  • 댓글 첨부파일일 경우

  • 댓글 table에 있는 해당 컬럼의 isDeleted 를 true로 변경

  • attachment table에 있는 해당 컬럼의 isDeleted를 true로 변경

  • 일반 댓글일 경우

  • 댓글 table에 있는 해당 컬럼의 isDeleted 를 true로 변경

+++ @joowojr @parkjaehak CT-181 commit들만 review해주시면 될 것 같습니다!

Requirements for Reviewer

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

PR Log

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

새롭게 배운 것

고민 중인 사항

첨부 자료

Requirements for Reviewer

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

✅ 체크리스트(Checklist)

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

🚪 이슈 번호(Issue numbers)

Closes #{이슈 번호}

starboxxxx and others added 21 commits January 30, 2025 23:23
@starboxxxx starboxxxx added ✨ feature 구현·개선 사항에 관련된 내용입니다 MID 우선순위 중 labels Jan 30, 2025
@starboxxxx starboxxxx self-assigned this Jan 30, 2025
joowojr and others added 24 commits January 31, 2025 03:23
# Conflicts:
#	src/main/java/clap/server/exception/code/MemberErrorCode.java
@parkjaehak
Copy link
Collaborator

soft delete 할때 로직으로 구현하는대신 jpa에서 간단한 애노테이션을 제공하는데 한 번 확인해주셔도 좋을 것 같습니다.
ref) https://velog.io/@max9106/JPA-soft-delete

@parkjaehak
Copy link
Collaborator

추가로 커밋 메세지의 경우 중복되거나 비슷한 내용일 경우 squash를 통해 합쳐서 커밋 해주시면 좋을 것 같습니다.

@starboxxxx
Copy link
Collaborator Author

soft delete 할때 로직으로 구현하는대신 jpa에서 간단한 애노테이션을 제공하는데 한 번 확인해주셔도 좋을 것 같습니다. ref) https://velog.io/@max9106/JPA-soft-delete

넵 이거 지금 rebate 과정에서 브랜치가 너무 난잡해져서 브랜치 새로파서 리팩토링 진행하겠습니다!!

@starboxxxx starboxxxx merged commit 38d1ad0 into develop Jan 31, 2025
1 check passed
@starboxxxx starboxxxx deleted the CLAP-181 branch January 31, 2025 08:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ feature 구현·개선 사항에 관련된 내용입니다 MID 우선순위 중

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants