Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

[#122] 프로필 수정 시 이름도 함께 수정하도록 변경 #123

Merged
merged 2 commits into from
Jul 3, 2024

Conversation

yel-m
Copy link
Member

@yel-m yel-m commented Jul 2, 2024

#️⃣ 연관된 이슈

Resolves #122

📝 작업 내용

이번 PR에서 작업한 내용을 간략히 설명해주세요 (이미지 첨부 가능)

  • 프로필 수정 시 이름도 함께 수정하도록 DTO와 updateUserProfile 메서드 로직을 수정했습니다.
  • build.gradle에서 사용하지 않는 의존성을 삭제했습니다.

/api/users/profile로 요청 시 다음과 같은 응답을 받을 수 있습니다.

{
  "success": true,
  "code": 0,
  "message": "유저 프로필을 성공적으로 수정했습니다.",
  "data": {
    "userId": 29,
    "name": "해피니스큐엘",
    "gender": "남성",
    "age": 22
  }
}

스크린샷 (선택)

💬 리뷰 요구사항(선택)

리뷰어가 특별히 봐주었으면 하는 부분이 있다면 작성해주세요

ex) 메서드 XXX의 이름을 더 잘 짓고 싶은데 혹시 좋은 명칭이 있을까요?

✅ Check List

  • PR 제목을 커밋 규칙에 맞게 작성했는가?
  • PR에 해당되는 Issue를 연결했는가?
  • 적절한 라벨을 설정했는가?
  • 작업한 사람을 모두 Assign했는가?

@yel-m yel-m self-assigned this Jul 2, 2024
@yel-m yel-m added the fix 기능 및 버그를 수정합니다 label Jul 2, 2024
@yel-m yel-m requested a review from KkomSang July 3, 2024 04:31
@KkomSang
Copy link
Collaborator

KkomSang commented Jul 3, 2024

짱짱 수고하셨습니😉

@KkomSang KkomSang merged commit f790f9f into develop Jul 3, 2024
1 check passed
@KkomSang KkomSang deleted the fix/#122 branch July 3, 2024 06:36
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
fix 기능 및 버그를 수정합니다
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[fix] 프로필 수정 시 이름도 함께 수정할 수 있도록 변경
2 participants