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] 회차 삭제 API 개발 #58

Merged
merged 7 commits into from
Jan 1, 2025
Merged

[Feat] 회차 삭제 API 개발 #58

merged 7 commits into from
Jan 1, 2025

Conversation

Moveuk
Copy link
Member

@Moveuk Moveuk commented Dec 25, 2024

📌 관련 이슈

closed #57

✨ PR 내용

  1. 회차 삭제 API 개발
  2. 조회 메서드 네이밍 변경

📚 레퍼런스 혹은 궁금한 사항들

회차 Meeting 삭제시 Fine, Attendance, Proof의 연관관계 및 FK로 인해 삭제가 불가능했었기에 지금은 OnDelete 어노테이션을 사용하여 해결한 상태입니다.

따라서, FK 기존의 로컬과 dev 서버의 db에서도 변화가 필요할 듯 합니다.

고려했던 해결책들

  1. 양방향 연관관계를 통해 Cascade를 적용시킨다.(OrphanRemoval 고려) <- ORM 레벨
  2. 단방향을 유지하고 OnDelete를 통해 Cascade를 적용한다. <- DB 레벨
  3. Applcation 서비스 내부에서 관련된 모든 테이블을 조회해서 삭제한다.

위 세가지를 고려했으나 변경점이 많고 상의해야할 부분이 많아질 것 같아 일단 비교적 간단한 2번을 선택했습니다.(추후 DB 세팅을 다시한다는 전제하에...)

3번도 간단하게 바로 적용이 가능하지만, 개발자가 매번 연관관계 매핑을 체크해야하기 때문에 개발 생산성이 떨어질 것으로 판단하여 일단 제외 했습니다.

1번의 경우에는 FK를 제외하여도 사용 가능하나 아직 DB에서 FK 제약조건을 걸 것인지, SoftDelete를 만들 것인지 등의 여러 이슈와 겹친다고 생각하여 일단 제외하고 기능 개발을 우선으로 진행했습니다.

📸 스크린샷(선택)

@Moveuk Moveuk added the enhancement New feature or request label Dec 25, 2024
@Moveuk Moveuk requested a review from uuujini December 25, 2024 15:30
@Moveuk Moveuk self-assigned this Dec 25, 2024
@Moveuk Moveuk linked an issue Dec 25, 2024 that may be closed by this pull request
1 task
@Moveuk Moveuk changed the base branch from main to develop December 25, 2024 15:30
Copy link
Member

@uuujini uuujini left a comment

Choose a reason for hiding this comment

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

고생하셨습니다~!👍
코멘트 확인해 주시면 감사하겠습니다!

@Moveuk Moveuk merged commit ed1cd6e into develop Jan 1, 2025
1 check passed
@Moveuk Moveuk deleted the meeting-delete branch January 1, 2025 07:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feat] 회차 삭제 API 개발
2 participants