Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

[✨ feat/#123] 마이페이지 > 프로필 정보 이미지 전달방식 변경 & 마이페이지 > 프로필 수정 기능 추가 #124

Merged
merged 12 commits into from
Sep 5, 2024

Conversation

junggyo1020
Copy link
Contributor

📄 Work Description

  • profileImage를 기존에 int 형 인덱스 값에서 ENUM 타입(String) 값으로 변경함에 따라 ENUM 파일을 새로 작성해 주었습니다.
  • 온보딩 > 회원가입 API에서 profileImage를 int 타입에서 String 타입으로 전달 받도록 변경했습니다.
    (6개의 String Enum 타입 중 하나: basic, lucky, smart, glass, calendar, passion)
    • 프로필 이미지가 null인 경우 기본값인 basic 으로 설정되도록 구현하였습니다.
  • 마이페이지 > 프로필 정보 불러오기 API에서 profileImage를 함께 보내주도록 response를 수정하였습니다.
    (6개의 String Enum 타입 중 하나: basic, lucky, smart, glass, calendar, passion)
  • 마이페이지 > 프로필 수정 기능을 추가했습니다.
    • 유효하지 않은 프로필 이미지로 수정을 시도할 경우, 예외처리 하여 CustomException을 발생시키도록 구현하였습니다.

⚙️ ISSUE

📷 Screenshot

1. 회원가입 API : profileImage 전달방식 변경 int >> String

image

2. 프로필 정보 불러오기 API : profileImage를 추가로 전달하도록 응답 수정

image

3. 프로필 수정하기 API

  • case 1) Swagger 200
image
  • case 2) Swagger 404 : 유효하지 않은 프로필 이미지로 수정을 시도할 경우
image

💬 To Reviewers

기존의 코드와 변경된 부분이 많고, 회원가입 로직에 코드를 추가했기 때문에 꼼꼼하게 확인해주시면 감사하겠습니다:)

잘못된 부분이 있거나 개선할 부분이 있다면 편하게 말씀해주세요!

🔗 Reference

문제를 해결하면서 도움이 되었거나, 참고했던 사이트(코드링크)

@junggyo1020 junggyo1020 added ✨ feat 새로운 기능 추가 ♻️ refactor 코드 리팩토링 ex) 형식변경 🐶정교🐶 labels Sep 4, 2024
@junggyo1020 junggyo1020 self-assigned this Sep 4, 2024
Copy link
Member

@JungYoonShin JungYoonShin left a comment

Choose a reason for hiding this comment

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

수고하셨습니다~! 주석을 깔끔하게 잘 써주셔서 코드 이해하기가 수월했습니다!

@junggyo1020 junggyo1020 merged commit 50282be into develop Sep 5, 2024
1 check passed
@junggyo1020 junggyo1020 deleted the feat/#123 branch October 11, 2024 00:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ feat 새로운 기능 추가 ♻️ refactor 코드 리팩토링 ex) 형식변경 🐶정교🐶
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants