[BE] feat: 관리자 학교 응답 누락된 필드 추가, 관리자 아티스트 응답 createdAt, updatedAt 필드 추가 (#849) #850
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
- logoUrl, backgroundImageUrl, createdAt, updatedAt
- createdAt, updatedAt
seokjin8678
added
BE
Test Results181 files 181 suites 30s ⏱️ Results for commit 3a5d1d9. ♻️ This comment has been updated with latest results. |
carsago
changed the title
carsago
approved these changes
Apr 11, 2024
# Conflicts: # backend/src/main/java/com/festago/artist/domain/Artist.java
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 세부 내용
이슈 내용 그대로 관리자 학교 응답인
AdminSchoolV1Response에 로고 URL과 백그라운드 이미지 URL, createdAt, updatedAt 필드를 추가했습니다.또한
AdminArtistV1Response에도 createdAt, updatedAt 필드를 추가하려고 했는데, Artist 엔티티에BaseTimeEntity가 상속되지 않아서BaseTimeEntity를 상속하게 했습니다.(DB에는 created_at, updated_at 컬럼이 있어서 충돌은 없습니다)
QueryDSL이 확실히 편하네요. 😂