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.

Sign up for GitHub

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

Jump to bottom

[BE] Artist를 추가할 때 중복된 Artist가 생기지 않도록 중복 검증을 추가한다. #963

Closed
seokjin8678 opened this issue May 13, 2024 · 0 comments · Fixed by #964
Closed

[BE] Artist를 추가할 때 중복된 Artist가 생기지 않도록 중복 검증을 추가한다. #963

seokjin8678 opened this issue May 13, 2024 · 0 comments · Fixed by #964
Assignees
@seokjin8678
Labels
ADMIN BE 백엔드에 관련된 작업 🏗️ 기능 기능 추가에 관한 작업

Comments

@seokjin8678
Copy link
Collaborator

✨ 세부 내용

관리자 페이지에서 Artist를 추가할 때, 기존 Artist가 있는데도, 새롭게 Artist를 추가하여 중복된 Artist가 생기는 이슈가 생겼습니다.

Artist는 사용자가 북마크가 가능하므로, 중복된 Artist가 생기면 혼란이 발생할 수 있습니다.

따라서 Artist를 생성하는 Service 레이어에 이름으로 중복을 검증하는 기능이 들어가면 좋을 것 같습니다.

⏰ 예상 소요 시간

30분
@seokjin8678 seokjin8678 added BE 백엔드에 관련된 작업 ADMIN 🏗️ 기능 기능 추가에 관한 작업 labels May 13, 2024
@seokjin8678 seokjin8678 self-assigned this May 13, 2024
@seokjin8678 seokjin8678 mentioned this issue May 13, 2024
Merged
@seokjin8678 seokjin8678 linked a pull request May 13, 2024 that will close this issue
[BE] feat: Artist 생성 시 중복 이름 검증 추가 (#963) #964
Merged
seokjin8678 added a commit that referenced this issue May 15, 2024
@seokjin8678
[BE] feat: Artist 생성 시 중복 이름 검증 추가 (#963) (#964)
f0e6f46 
feat: Artist 생성 시 중복 이름 검증 추가
@github-project-automation github-project-automation bot moved this from Todo to Done in 2023-festa-go May 15, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@seokjin8678 seokjin8678

Labels
ADMIN BE 백엔드에 관련된 작업 🏗️ 기능 기능 추가에 관한 작업
Projects
2023-festa-go
Status: Done
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

[BE] feat: Artist 생성 시 중복 이름 검증 추가 (#963) woowacourse-teams/2023-festa-go
1 participant
@seokjin8678