Skip to content

Conversation

@leesewon00
Copy link
Member

@leesewon00 leesewon00 commented Aug 25, 2024

관련 이슈

작업 내용

  • 프로필 이미지 수정 API 개발

  • 닉네임 수정 API 개발

  • 테스트코드 작성하였습니다.

  • 포스트맨에 API 등록했습니다.

image

특이 사항

추후 닉네임과 프로필 이미지를 동시에 수정하는 api는 삭제할 예정입니다.

추후 기본 프로필 이미지가 도입되면 프로필 이미지 삭제 기능을 도입하여 기본 프로필 이미지를 등록시킬 수 있습니다.

리뷰 요구사항 (선택)

@wibaek
Copy link
Member

wibaek commented Aug 25, 2024

lgtm

@wibaek
Copy link
Member

wibaek commented Aug 25, 2024

#67 PR 머지될 때 Squash and merge가 안되있어서, PR 머지시에 Squash and merge, release 머지시에 일반 Create a merge commit 체크해주시면 감사하겠습니다!

@leesewon00
Copy link
Member Author

닉네임과 프로필 이미지를 동시에 수정하는 기존의 마이페이지 정보 수정 API 삭제하였습니다.

@leesewon00 leesewon00 merged commit 9062f81 into solid-connection:main Aug 25, 2024
@nayonsoso
Copy link
Collaborator

저 궁금한 것이 있습니다🙋🏻‍♀️

  1. 프로필 사진 수정과 닉네임 수정 api 를 분리하신 이유가 있으신가요? 둘 중 하나만 성공하고 하나만 실패했을 때의 상황이 우려되어서요!

  2. 만약 그런 상황이 생긴다면 일부분만 수정된 결과를 시용자는 보게 되는건가요?

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.

프로필 이미지 수정, 닉네임 수정 API 분리

3 participants