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

[fix] #243 - 공연 생성 및 수정 시 performancePeriod, scheduleNumber 부여 로직 추가 #244

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

hyerinhwang-sailin
Copy link
Collaborator

@hyerinhwang-sailin hyerinhwang-sailin commented Nov 4, 2024

Related issue 🛠

Work Description ✏️

  • 공연 생성, 수정 시 request body의 schduleList의 회차날짜를 기준으로 performancePeriod를 생성하고, schduleNumber를 부여하도록 코드를 수정했습니다.
  • 생성과 수정 모두에 사용되는 코드이기에 재사용을 위해 performance에 update 비즈니스 로직을 작성했습니다.

Trouble Shooting ⚽️

Related ScreenShot 📷

공연 생성

image
  • performanceDate들을 이용해 performancePeriod가 2024.11.10~2024.11.21로 정상적으로 부여된 것을 확인했습니다.
  • request에서 schduleNumber를 잘못 작성한 경우에도 schduleNumber가 다시 정상적으로 부여된 것을 확인했습니다.

공연 수정

image
  • 수정된 performanceDate들을 이용해 performancePeriod가 2024.11.06~2024.12.31로 정상적으로 부여된 것을 확인했습니다.
  • 수정된 회차 날짜에 맞게 schduleNumber가 다시 정상적으로 부여된 것을 확인했습니다.

Uncompleted Tasks 😅

To Reviewers 📢

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

Successfully merging this pull request may close these issues.

[bug] 공연 회차 수정 시 performancePeriod에 반영하도록 수정
1 participant