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

[REFACTOR]/#85 - Converter와 ResponseDTO들 리팩토링 #99

Merged
merged 1 commit into from
Apr 10, 2024

Conversation

koojun99
Copy link
Contributor

@koojun99 koojun99 commented Apr 10, 2024

🚀 개요

회원가입, 로그인 관련 로직에 참조되는 Converter 클래스들을 static으로 선언
회원가입, 로그인 관련 로직에서 오고가는 DTO들의 구조를 API 명세서에 맞게 수정

🔍 변경사항

  • 닉네임 중복확인 API URL 변경
  • 트리하우스 초대 수락/거부 API 응답 DTO 변경
  • 트리하우스 멤버 가입 API 응답 DTO 변경
  • 내가 받은 초대장 조회 API 응답 DTO 변경

⏳ 작업 내용

  • 닉네임 중복확인 API URL 변경
  • 트리하우스 초대 수락/거부 API 응답 DTO 변경
  • 트리하우스 멤버 가입 API 응답 DTO 변경
  • 내가 받은 초대장 조회 API 응답 DTO 변경

📝 논의사항

내가 받은 초대장 조회 API (InvitationService)의 getInvitations 메서드에 초대장이 발신된 트리하우스 속 멤버들의 프로필 이미지를 포함시키는 로직이 있는데 개선의 여지가 있어보인다면 어떻게 고치는 게 좋을 지 궁금합니다!

@koojun99 koojun99 added the ♻️ refactor Refactoring code label Apr 10, 2024
@koojun99 koojun99 requested a review from CYY1007 April 10, 2024 09:59
@koojun99 koojun99 self-assigned this Apr 10, 2024
@CYY1007 CYY1007 merged commit a2cbdc5 into develop Apr 10, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ refactor Refactoring code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🔨[refactor] 회원가입/로그인 관련 Converter와 ResponseDTO들 리팩토링
2 participants