Skip to content

Conversation

@m-a-king
Copy link
Collaborator

@m-a-king m-a-king commented Feb 7, 2025

resolved :

📌 과제 설명

이미지 업로드

👩‍💻 요구 사항과 구현 내용

프로필 수정에서 이미지를 분리했습니다.
공통 이미지 업로드 컨트롤러도 추가했습니다.
서진 님이 작성해주신 기존의 프로필 이미지 수정을 유저 프로필 이미지 수정으로 병합했습니다.

✅ PR 포인트 & 궁금한 점

@m-a-king m-a-king requested review from 7zrv, ayoung-dev and leebs0521 and removed request for 7zrv February 7, 2025 05:46
@m-a-king m-a-king self-assigned this Feb 7, 2025
@m-a-king m-a-king changed the title Feature/344 add update volunteer profile image [FEATURE] 이미지 업로드 (프로필, 공통) Feb 7, 2025
@github-actions github-actions bot changed the title [FEATURE] 이미지 업로드 (프로필, 공통) [BUILD FAIL] Feature/344 add update volunteer profile image Feb 7, 2025
@github-actions github-actions bot closed this Feb 7, 2025
github-actions[bot]

This comment was marked as resolved.

@m-a-king m-a-king reopened this Feb 7, 2025
@m-a-king m-a-king changed the title [BUILD FAIL] Feature/344 add update volunteer profile image [FEATURE] 이미지 업로드 (프로필, 공통) Feb 7, 2025
github-actions[bot]

This comment was marked as resolved.

@github-actions github-actions bot changed the title [FEATURE] 이미지 업로드 (프로필, 공통) [BUILD FAIL] [FEATURE] 이미지 업로드 (프로필, 공통) Feb 7, 2025
@github-actions github-actions bot closed this Feb 7, 2025
@m-a-king m-a-king reopened this Feb 7, 2025
@m-a-king m-a-king changed the title [BUILD FAIL] [FEATURE] 이미지 업로드 (프로필, 공통) [FEATURE] 이미지 업로드 (프로필, 공통) Feb 7, 2025
@m-a-king m-a-king changed the title [FEATURE] 이미지 업로드 (프로필, 공통) [FEATURE] 이미지 업로드 (프로필, 공통) Feb 7, 2025
Copy link
Collaborator

@ayoung-dev ayoung-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생 많으셨습니다!

@RequestBody CenterBasicInfoRequestDto centerBasicInfoRequestDto
) {
updateBasicInfoUseCase.update(userId, centerBasicInfoRequestDto);
return ApiResponse.ok("센터 기본 정보 업데이 완료");
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

업데이..!
중요한 건 아니지만.. 보였어요

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 감사합니다 ㅋㅋㅋㅋ

Copy link
Collaborator

@leebs0521 leebs0521 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

고생하셨습니다.

public ApiResponse<String> getImageUploadUrl(
@UserId UUID userId,
@RequestBody String fileName
) {
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ReqeustBody에 ImgUrlRequestDto가 아닌 스트링으로 하신 이유가 있을까요??

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

아니요 이거 변경하겠습니다!

@m-a-king m-a-king dismissed github-actions[bot]’s stale review February 7, 2025 06:35

리베이스 문제 해결

@sonarqubecloud
Copy link

sonarqubecloud bot commented Feb 7, 2025

@m-a-king m-a-king merged commit c471a49 into main Feb 7, 2025
3 checks passed
@m-a-king m-a-king deleted the feature/344-add-update-volunteer-profile-image branch February 7, 2025 06:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants