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

[Feature] 모집자 프로필 - 기본 정보 조회 API 구현 #191

Merged
merged 3 commits into from
May 11, 2024

Conversation

hojeong2747
Copy link
Member

💡 연관된 이슈

close #190

📝 작업 내용

  • 모집자 프로필 - 기본 정보 조회 API 구현
  • 모집자 프로필 - 기본 정보 조회 API 테스트 코드 작성

💬 리뷰 요구 사항

모집자 프로필 후기 조회 시 이동봉사자 & 모집자를 분리해서 API 구현했던 것처럼, 모집자 프로필 기본 정도 조회도 마찬가지로 분리하여 구현했습니다. (코넥독 1.0에서는 해당 API는 없었기에 추가 개발입니다 :)

@hojeong2747 hojeong2747 added ✨ Feature 기능 개발 Priority: Medium 우선순위 중간 🐰 Hojeong 담당자 labels May 11, 2024
@hojeong2747 hojeong2747 requested a review from kyeong-hyeok May 11, 2024 02:55
Copy link
Member

@kyeong-hyeok kyeong-hyeok 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

@kyeong-hyeok kyeong-hyeok left a comment

Choose a reason for hiding this comment

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

코멘트 부분만 한 번 검토해 주세요!

, description = "M2, 해당 이동봉사 중개를 찾을 수 없습니다."
, content = @Content(schema = @Schema(implementation = ErrorResponse.class)))
})
@GetMapping("/intermediaries")
Copy link
Member

Choose a reason for hiding this comment

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

uri에서 자신의 프로필을 조회한다는 의미를 담으면 더 좋을 것 같습니다!

Copy link
Member Author

Choose a reason for hiding this comment

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

오홍 동의합니다! me, my 중 고민했는데 my로 결정했습니다. 이유는 다음과 같아요!

  • 이동봉사자의 마이페이지 정보와 모집자 프로필의 정보가 동일하게 '사용자의 기본 정보'를 가져옵니다.
  • 프론트 관점에서도 마이페이지/사용자의 프로필에서 사용하는 정보를 가져올 때 동일하게 /my uri로 통일되어 있는 게 구현할 때 안 헷갈릴 것 같습니다.

@hojeong2747 hojeong2747 merged commit c84343e into develop May 11, 2024
1 check passed
@hojeong2747 hojeong2747 deleted the feat/190-intermediary-info-api branch May 11, 2024 03:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
✨ Feature 기능 개발 🐰 Hojeong 담당자 Priority: Medium 우선순위 중간
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Feature] 모집자 프로필 - 기본 정보 조회 API 구현
2 participants