[FEAT] Album API 추가 기능 구현 #139
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
jun02160
added
🔥 Pull Request
ddongseop
approved these changes
Mar 10, 2024
| return albumList.stream() | ||
| .map(AlbumResponseDto::of) | ||
| .collect(Collectors.toList()); | ||
| } | ||
|
|
||
| private Album createAlbumExample() { | ||
| return new Album(0L, "사진의 제목을 입력할 수 있어요", "사진에 대해 소개해요", | ||
| "imgUrl", "직성자"); |
There was a problem hiding this comment.
혹시 이부분의 imgUrl이 ALBUM_EXAMPLE과 뭐가 다른것일까요!
There was a problem hiding this comment.
ALBUM_EXAMPLE 값이 존재하는 이유는 아래와 같이 정리할 수 있습니다!
- Example 객체의 경우, 버킷에서 이미지를 삭제하지 않습니다.
- 별도의 메서드로 분리할 필요 없이 하나의 메서드로 사용하기 위해 반환값인 문자열을 Controller에 넘겨줘야 했습니다.
*테이블의 한 row를 차지하지 않고 객체 상으로 관리하는 Album 값이기에 위와 같이 구분점을 두어 구현했습니다 !!
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.
📌 관련 이슈
closed #138
✨ 어떤 이유로 변경된 내용인지
최대 업로드 수 제한
Album 추가 등록 시, 최대 허용 가능한 업로드 수를 제한하는 방식으로 변경했습니다.
-> 해당 HTTP Status는 예외처리 편의성을 고려하여 501 Not Implemented로 처리했습니다!
튜토리얼용 Example Album 추가
Parentchild 테이블에
isFirstAlbumUpload,isDeleteSampleAlbum라는 필드를 추가하여 분기처리 하고, 아직 한번도 앨범을 등록하지 않은 경우라면, sample 데이터 (id 0번)를 리스트에 담아서 응답하도록 구현했습니다.🙏 검토 혹은 리뷰어에게 남기고 싶은 말