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

[FIX] 마이페이지 API 수정 #54

Merged
merged 7 commits into from
Jan 14, 2024
Merged

[FIX] 마이페이지 API 수정 #54

merged 7 commits into from
Jan 14, 2024

Conversation

eeddiinn
Copy link
Contributor

@eeddiinn eeddiinn commented Jan 13, 2024

📌 관련 이슈

closed #53

✨ 어떤 이유로 변경된 내용인지

  • 마이페이지 레큐북, 레큐노트에 favoriteName 을 추가하였습니다.
  • 마이페이지 레큐북에만 userNickname 이 있었는데 레큐노트가 디폴트 값이여서 레큐노트에도 userNickname을 추가하였습니다.
  • 마이페이지 레큐북에서 배경 색상을 반영 안해도 된다고 해서 bookBackgroundColor를 삭제하였습니다.
  • Swagger 설정한 부분을 수정하였습니다.

🙏 검토 혹은 리뷰어에게 남기고 싶은 말

  • 다들 푹 쉬구 아픈거 빨리 나았으면 좋겠어유 ..!! 🤗🤗

@eeddiinn eeddiinn added 예진❄️ 🔧Modify 코드 수정 (기능의 변화가 있을 때) 🔥 Pull Request PR 날림 labels Jan 13, 2024
@eeddiinn eeddiinn self-assigned this Jan 13, 2024
@ddongseop ddongseop changed the title [Fix] 마이페이지 수정 [FIX] 마이페이지 수정 Jan 14, 2024
Copy link
Member

@ddongseop ddongseop left a comment

Choose a reason for hiding this comment

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

꼼꼼하게 잘 바꿔주셨네요! 수고하셨습니당 :)

Copy link
Member

Choose a reason for hiding this comment

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

[P2] 오잉 이건 뭐징 ㅋㅋㅋ 삭제하구 커밋해주세요!

Copy link
Contributor

@dong2ast dong2ast left a comment

Choose a reason for hiding this comment

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

고생하셨습니다 ~~ 👍 👍

@@ -123,12 +119,12 @@ public ApiResponse<MypageBookResponseDto> getMypageBook(Principal principal) {
@GetMapping("/mypage/note")
@io.swagger.v3.oas.annotations.responses.ApiResponse(
responseCode = "200",
description = "마이페이지 내 레큐노트 조회에 성공했습니다.",
description = "마이페이의 유저 닉네임과 내 레큐노트 조회에 성공했습니다.",
Copy link
Contributor

Choose a reason for hiding this comment

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

[P3] 마이페이지로 수정 해주세요~

@ddongseop ddongseop merged commit 2a74407 into develop Jan 14, 2024
1 check passed
@ddongseop ddongseop deleted the fix/#53-mypage_fix branch January 14, 2024 19:22
@ddongseop ddongseop changed the title [FIX] 마이페이지 수정 [FIX] 마이페이지 API 수정 Jan 14, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
예진❄️ 🔧Modify 코드 수정 (기능의 변화가 있을 때) 🔥 Pull Request PR 날림
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FIX] 마이페이지 데이터 수정
3 participants