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

[REFACTOR] 같은 멤버가 등록중인 노래에 같은 부분을 킬링파트로 등록시 추가 데이터를 저장하지 않는다. #461

Closed
4 tasks
splitCoding opened this issue Sep 26, 2023 · 0 comments · Fixed by #469
Assignees
Labels
[ 🌙 BE ] 백엔드 크루들의 멋진 개발 이야기 하나둘셋 야! 🔨 Refactor 꾸준한 개선이 더 나은 애플리케이션을 만든다

Comments

@splitCoding
Copy link
Collaborator

splitCoding commented Sep 26, 2023

🤷 개선할 기능

기존 멤버가 등록중인 노래에 같은 부분을 킬링파트로 등록시 추가 데이터를 저장하지 않는다.

🔨 상세 작업 내용

  • 등록 파트에 동록한 회원 정보를 추가한다.
  • 파트 등록 요청이 들어오면 해당 파트가 등록되어있는지 확인한다.
  • 중복 파트일 경우 상태 코드 200(OK) 로 응답한다.
  • 새로운 파트일 경우 상태 코드 201(CREATED) 로 응답한다.

⏰ 예상 작업 시간

2시간

⏰ 마감일

230926

@splitCoding splitCoding added [ 🌙 BE ] 백엔드 크루들의 멋진 개발 이야기 하나둘셋 야! 🔨 Refactor 꾸준한 개선이 더 나은 애플리케이션을 만든다 labels Sep 26, 2023
@splitCoding splitCoding self-assigned this Sep 26, 2023
splitCoding added a commit that referenced this issue Sep 27, 2023
…469)

* refactor: VotingSongPart 에 ManyToOne 관계의 Member 필드 추가

생성자 변경으로 인해 실패하는 테스트 수정, sql 스크립트에 해당 변경 내용 적용

* feat: 멤버가 등록한 같은 파트가 있는지 확인하는 메서드 추가

* feat: VotingSongPartService 멤버가 등록한 같은 파트 처리 로직 추가

* feat: VotingSongPartController 멤버가 등록한 같은 파트 처리 로직, 테스트 추가

* style: 공백 제거

* refactor: VotingSongPart 의 Member 필드를 삭제하고 Vote 에 멤버를 추가한다.

* refactor: VotingSongPartService 로직 변경

* refactor: 리뷰 반영
@github-project-automation github-project-automation bot moved this from Todo to Done in 2023-shook Sep 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
[ 🌙 BE ] 백엔드 크루들의 멋진 개발 이야기 하나둘셋 야! 🔨 Refactor 꾸준한 개선이 더 나은 애플리케이션을 만든다
Projects
Status: Done
1 participant