-
Notifications
You must be signed in to change notification settings - Fork 8
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: 소셜미디어 CRUD 기능 추가 (#802) #848
Merged
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- validate -> validateCreate
- findByOwnerIdAndOwnerType: 목록 조회 - findById: 단건 조회
seokjin8678
added
BE
백엔드에 관련된 작업
ADMIN
📬 API
API가 변경되거나 추가되는 작업
🏗️ 기능
기능 추가에 관한 작업
labels
Apr 8, 2024
Test Results197 files 197 suites 32s ⏱️ Results for commit 8064e16. ♻️ This comment has been updated with latest results. |
- ownerId, ownerType
관리자 프론트 페이지에 기능 완성 되었고, 추가 리뷰 없다고 판단하여 머지하도록 하겠습니다! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
📌 관련 이슈
✨ PR 세부 내용
관련 이슈처럼 소셜미디어 CRUD 기능을 추가했습니다.
이슈에서는 CUD 기능만 추가하기로 되어 있지만, 관리자 페이지에 빠르게 기능 구현이 필요하고, 구현의 어려움이 없으므로 해당 PR에서 작업하였습니다.
소셜미디어의 추가와 수정에 문자열의 길이에 대한 검증 로직이 없는데, 관리자 기능이라 추가하지는 않았습니다.
(추후 500 에러가 발생하는 것을 보고 추가하면 될 것 같네요)
소셜미디어 조회 API는 2개인데, 그중 식별자를 통한 단건 조회 기능은 수정 기능에서 사용할 목적으로 만들었습니다. (상세 조회)
목록 조회는
ownerId
,ownerType
에 해당하는 소셜미디어를 모두 조회해 오는데, 따로 limit을 두지는 않았습니다.이유는
ownerId
와ownerType
의 조합에서 생성할 수 있는 소셜미디어의 최대 개수는socialMediaType
개수와 동일하기에, 너무 많은 row로 인해 발생하는 성능 이슈가 없습니다.SocialMediaType.values().length
를 limit 절에 사용한다면 나름 성능 최적화가 가능할 것 같네요. 다만 모든 socialMediaType을 만드는 경우가 없을 것 같아 현실성은 떨어져 보입니다. 😂또한
SocialMediaCommandService
의 TODO 주석에서 남겨두었지만, 추상화된 예외가 필요할지 얘기하고 싶습니다.더 객체 지향적인 방법으로 풀어나간다면 SocialMediaCreateValidator 인터페이스를 만들어서 어떻게 할 것 같은데...
현재 소셜미디어의 owner는 School과 Artist 2개뿐이고, 여기서 더 추가될 것 같지는 않아서 그저 오버 엔지니어링이 될 것 같네요. 😂
그 외, 메서드 이름이 적절한지 봐주시면 감사하겠습니다!