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

[BE] feat: StageTicket 생성, 삭제 로직 추가 (#1007-2) #1009

Open
wants to merge 1 commit into
base: feat/#1007-1
Choose a base branch
from

Conversation

seokjin8678
Copy link
Collaborator

📌 관련 이슈

✨ PR 세부 내용

#1008 PR에 의존하는 다음 StageTicket 생성, 삭제 로직에 대한 PR입니다.

티켓을 생성, 삭제하면 TicketCreatedEvent, TicketDeletedEvent를 발행합니다.

생성, 삭제 기능만 있고 수정 기능은 없는데, 굳이 티켓을 수정할 필요가 없더군요. 😂

또한 티켓의 생성과 삭제는 공연의 ticketOpenTime 이후로 동작하지 않도록 했습니다.

공연의 수정과 삭제 또한 등록된 티켓이 있으면 동작하지 않도록 했습니다.

이는 티켓팅 시 발생하는 정합성의 문제를 해결하기 위함입니다.
(정합성 외에도 티켓 오픈 시간 이후 티켓 수량을 변경하는 일이 일어나면 논란이 생길 것 같네요 😂)

DB에는 티켓의 정보가 반영되지만, 레디스에서는 반영이 되지 않기 때문에 해당 변경을 반영하는 작업이 필요합니다.

해당 반영은 위에서 발행한 이벤트를 가지고 처리합니다.

해당 코드는 이후 PR에 있으니 참고하시면 될 것 같습니다.

그 외 의도했던 내용에 대해선 코드에 리뷰로 남기겠습니다!

@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 🏗️ 기능 기능 추가에 관한 작업 labels Jun 11, 2024
@seokjin8678 seokjin8678 self-assigned this Jun 11, 2024
Copy link

Test Results

256 files  +2  256 suites  +2   30s ⏱️ ±0s
843 tests +8  843 ✅ +8  0 💤 ±0  0 ❌ ±0 
868 runs  +8  868 ✅ +8  0 💤 ±0  0 ❌ ±0 

Results for commit 29f42c5. ± Comparison against base commit 9503869.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
BE 백엔드에 관련된 작업 🏗️ 기능 기능 추가에 관한 작업
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant