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

건물 리뷰 리스트 조회 엔드포인트 변경에 관한 논의 #23

Open
CODe5753 opened this issue Aug 29, 2022 · 0 comments
Open

Comments

@CODe5753
Copy link
Member

What

image

/api/v1/review/detail의 엔드포인트 계층이 모호하다는 Client쪽 의견이 있었습니다. (슬랙 참고)

How

/api/v1/review/detail/api/v1/building/{buildingId}/reviews로 변경하는 것이 어떻냐는 의견을 함께 받았습니다.

Topic

아직 자세히 살펴보진 않았으나 PathVariable을 사용할 때 항상 이 점들을 고려합니다.

  • PathVariable 사용으로 인해 다른 URI와 혼선은 없는지(엔드포인트 충돌)
  • 이 URI로 충분한 의미를 전달하는 것이 맞는지

의견 주시면 감사하겠습니다!

@CODe5753 CODe5753 changed the title 건물 리뷰 리스트 조회 엔드포인트 변경에 관해 건물 리뷰 리스트 조회 엔드포인트 변경에 관한 논의 Aug 29, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant