[FEAT] 기록하기(Album) API 구현 #135
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
Feb 29, 2024
| } | ||
|
|
||
| // PreSigned Url 이용 (클라이언트에서 해당 URL로 업로드) | ||
| @PatchMapping("/image") |
There was a problem hiding this comment.
이 부분 PATCH인것 확인후에 명세서에 반영해뒀습니다!
| public ApiResponse createAlbum(@Valid @RequestBody final CreateAlbumRequestDto request, final Principal principal, HttpServletResponse response) { | ||
| String imgUrl = s3Service.getS3ImgUrl(ALBUM_PREFIX.getValue(), request.getImgFileName()); | ||
| Long albumId = albumService.createAlbum(request, imgUrl, getUserFromPrincial(principal)); | ||
| response.setHeader("Location", "/album/" + albumId); |
There was a problem hiding this comment.
Q) 혹시 header를 지정함으로써 해서 얻는 효과가 무엇일까요!
There was a problem hiding this comment.
HTTP 통신 규약에 따르면 201 Created로 응답하는 경우에 생성된 리소스의 값을 응답에 포함시키는 게 원칙이라고 하더라구요! 그래서 응답 헤더(Location)에 생성된 리소스의 식별자를 요청 url에 붙여서 반환하도록 HttpServleResponse를 인자로 받아 넣어 주었습니다!!
이 부분은 다른 API에도 추후 적용해보면 좋을 것 같네요 :)
*HTTP 201 Created에 관한 규약 아래 내용 참고해 주세요!!
https://developer.mozilla.org/ko/docs/Web/HTTP/Status/201
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 #133
✨ 어떤 이유로 변경된 내용인지
Album 엔티티 설계
회의 때 논의한대로 DB 작업을 진행했고, Parentchild과의 양방향 연관관계를 가지도록 매핑했습니다.
*논의사항에서 변경된 내용은 작성자 필드명을
username이 아닌writer로 지정한 점입니다!API 개발
SP3에 반영된 '기록하기' 페이지에 대한 API를 개발했습니다.
-> 기록하기의 각 '사진+글'을 Album(앨범) 으로 지칭하고 명세서와 로직을 구성했습니다.
위 4개의 API를 추가했고, 이미지 파일 업로드는 S3 PreSigned Url을 이용해 구현했습니다.
🙏 검토 혹은 리뷰어에게 남기고 싶은 말
🧡엄빠 화이팅🧡