Skip to content

Feat/203 일정 삭제 정책 수정#212

Closed
jaylovegood wants to merge 28 commits intodevfrom
feat/203-일정-삭제-정책-수정

Hidden character warning

The head ref may contain hidden characters: "feat/203-\uc77c\uc815-\uc0ad\uc81c-\uc815\ucc45-\uc218\uc815"
Closed

Feat/203 일정 삭제 정책 수정#212
jaylovegood wants to merge 28 commits intodevfrom
feat/203-일정-삭제-정책-수정

Conversation

@jaylovegood
Copy link
Collaborator

✨ Feature PR (to dev)

중요: Feat/202 PR을 먼저 리뷰 후 머지해주세요!!!!!!! 그 브랜치 위에 쌓은 거라서 base를 dev가 아닌 202 브랜치로 해놓았습니다. 202 먼저 머지하면 이 PR의 베이스가 자동으로 dev로 변경됩니다

🧪 로컬 테스트 여부 (작업자 체크)

  • 로컬에서 Swagger 혹은 테스트 코드로 동작을 확인했습니다.

📄 documentation 최신화 여부

변경사항과 관련된 API의 swagger documentation이 실제 동작과 일치하는지 확인합니다.

  • (작업자) 확인하였습니다.
  • (리뷰어) 확인하였습니다.

📌 작업 내용(what & why)

  • 삭제 로직 (EventService.delete())

    • CONFIRMED + WAITLISTED 신청자만 이메일 알림 대상 (BANNED 제외)
    • 삭제 전에 유저 정보 조회 → 이메일 데이터 구성
    • registrations 먼저 삭제 → event 삭제 (FK 제약)
    • afterCommit에서 이메일 발송 (트랜잭션 커밋 후 실행)
    • Pessimistic Lock으로 동시성 처리
  • 이메일 (EmailService + event-cancelled.html)

    • sendEventCancellationEmail() 메서드 추가
    • HTML 템플릿: 제목/일시/장소/주최자 연락처/상세설명 포함
  • 에러코드 정리

    • CANNOT_DELETE_EVENT_WITH_REGISTRATIONS 에러코드 및 예외 제거

🔎 영향 범위

  • 동시성 / 트랜잭션 처리 영향

🔥 관련 이슈

jaylovegood and others added 24 commits March 15, 2026 19:46
@jaylovegood jaylovegood marked this pull request as draft March 17, 2026 00:13
@jaylovegood jaylovegood force-pushed the feat/203-일정-삭제-정책-수정 branch from c4f9dba to a57f168 Compare March 17, 2026 00:18
@jaylovegood
Copy link
Collaborator Author

base branch 바꾸기 전 찬규님 리뷰
#209 (comment)

@jaylovegood
Copy link
Collaborator Author

데드락 방지를 위해 어디에서든 event -> registration 순으로 잠그는 것으로 통일하겠습니다!

@jaylovegood jaylovegood marked this pull request as ready for review March 17, 2026 00:33
Copy link
Collaborator

@uykhc uykhc left a comment

Choose a reason for hiding this comment

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

확인했습니다. 수고하셨습니다!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

일정 삭제 정책 수정

2 participants