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 : 소프트 딜리트 작성 #310

Merged
merged 1,253 commits into from
Apr 28, 2024
Merged

Feat : 소프트 딜리트 작성 #310

merged 1,253 commits into from
Apr 28, 2024

Conversation

mingmingmon
Copy link
Collaborator

Summary

소프트 딜리트를 필요한 domian들에 적용했습니다.

아래와 같은 주요사항이 있습니다.

  1. 댓글 삭제의 경우 "삭제된 댓글입니다"를 프론트에서 표현하기 위해 첨부된 사진을 참고해주세요.
  • 부모 댓글이 삭제된 경우 모든 정보를 null로 반환하고, 그 아래 자식 댓글의 내용은 전부 반환합니다.
  • 삭제된 댓글에 또 다시 대댓글을 달 수도 있습니다.
  • SUPER 계정은 삭제된 댓글을 게시글(Board)의 id를 통해 검색해서 볼 수 있고, 이때 삭제된 댓글이 가지고 있는 자식 댓글은 표현하지 않습니다.
  1. 활동그룹 게시판의 경우는 부모 게시판 삭제 시 자식 게시판도 함께 삭제됩니다. 활동그룹 게시판에서는 부모 게시판이 삭제되면 자식 게시판도 삭제되는 것이 옳다고 판단했기에, 댓글처럼 따로 "삭제된 게시판입니다"를 표현하지 않아도 되기 때문입니다.
  2. Book 엔티티는 계속해서 column index out of range 2: 1 이라는 오류가 발생하여 SQLDelete를 사용하지 않고 직접 isDeleted 변수를 update하는 방식으로 진행했습니다.

Tasks

  • SQLDelete와 SQLRestriction을 사용소프트 딜리트 적용
  • Comment의 경우 "삭제된 댓글입니다" 표현을 위한 커스텀 소프트 딜리트 적용
  • Book의 경우 DB 컬럼 이슈 때문에 커스텀 소프트 딜리트 적용

Screenshot

  • 부모 댓글 3이 지워진 경우 isDeleted : true, 다른 정보들은 null로 반환, 그러나 자식 댓글 4,5는 지워지지 않았으므로 isDeleted : false, 다른 정보들 담아서 정상 반환
    image

  • 삭제된 댓글을 게시글로 검색해서 볼 수 있고, 자식 댓글을 표현하지 않음
    image

limehee added 30 commits March 16, 2024 21:42
@mingmingmon mingmingmon requested a review from limehee as a code owner April 27, 2024 13:23
@mingmingmon mingmingmon self-assigned this Apr 27, 2024
@mingmingmon mingmingmon added the ✨ Feature 새로운 기능 명세 및 개발 label Apr 27, 2024
@limehee limehee merged commit 978c929 into refactor/#299 Apr 28, 2024
1 check failed
@limehee limehee deleted the feat/entityDelete branch April 28, 2024 06:25
@limehee limehee restored the feat/entityDelete branch April 28, 2024 06:28
@limehee limehee deleted the feat/entityDelete branch April 28, 2024 08:35
limehee added a commit that referenced this pull request Apr 28, 2024
Feat : 소프트 딜리트 작성
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 새로운 기능 명세 및 개발
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants