Refactor/339 기관 프로필 이미지 업데이트 리팩토링 #348
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
- userId 직렬화 어노테이션 변경 - 테스트및 검증 완료
- 프로필 이미지 url 필드를 변경하는 메서드 구현 - 테스트및 검증 완료
- 프로필 이미지 업데이트 유스케이스및 구현체 구현 - 요청 레코드 생성 - 테스트및 검증 완료
- 프로필 이미지 업데이트 유스케이스및 구현체 구현 - 요청 레코드 생성 - 테스트및 검증 완료
- 프로필 이미지 업데이트 엔드 포인트 구현 - 테스트및 검증 완료
2 tasks
7zrv
requested review from
leebs0521,
m-a-king and
ayoung-dev
and removed request for
leebs0521 and
m-a-king
|
m-a-king
approved these changes
Feb 4, 2025
|
|
||
| String presignedUrl = imageUploadUseCase.getPresignedUrl(requestDto.fileName()); | ||
|
|
||
| String fileUrl = presignedUrl.split("\\?")[0]; |
There was a problem hiding this comment.
presignedUrl을 클래스로 만들고, 스플릿 메서드를 메서드로 나타내도 좋을 것 같습니다
leebs0521
approved these changes
Feb 4, 2025
| @Secured("ROLE_CENTER") | ||
| @Operation(summary = "기관 프로필 이미지 수정", description = "기관의 프로필 이미지를 수정합니다.") | ||
| @PutMapping("/profileImgUpdate") | ||
| public ApiResponse<String> updateCenterProfile( |
There was a problem hiding this comment.
카멜케이스보다 /profile-image 이나 /profile/image 정도로 바꾸는게 좋아보입니다.
| @@ -34,9 +34,9 @@ public class PreferItemCommandApiController { | |||
| @PostMapping() | |||
| public ApiResponse<PreferItemCreateResponseDto> registerPreferItem( | |||
| @Valid @RequestBody PreferItemCreateRequestDto requestDto, | |||
| @CurrentUser UUID userId) { | |||
| @UserId UUID centerId) { | |||
|
|
|||
| public record UpdateProfileImgUrlRequestDto( | ||
| UUID userId, | ||
| String profileImgUrl | ||
| ) { |
m-a-king
reviewed
Feb 4, 2025
Comment on lines
+5
to
+7
| public record CenterProfileImgUpdateRequestDto( | ||
| @Schema(description = "파일 이름", example = "image.png") | ||
| String fileName |
There was a problem hiding this comment.
. 이 포함되어야하는 검증이 추가돼도 될 것 같습니다
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.
📌 과제 설명
이미지 업로드 방식이 presigned 방식으로 변경됨에 따라
기관 프로필 이미지 업데이트 api를 리팩토링했습니다.
프론트에서의 에러가 우려되어 기존 이용하던 기능은 그대로 두었습니다.
👩💻 요구 사항과 구현 내용
✅ PR 포인트 & 궁금한 점