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

[ALL] RESTful API 디자인 원칙에 기반하여 기존 API path를 수정 #107

Merged
merged 2 commits into from
Jul 30, 2024

Conversation

seokmyungham
Copy link
Contributor

@seokmyungham seokmyungham commented Jul 29, 2024

관련 이슈

작업 내용

RESTful API 디자인 원칙에 기반하여 현재 구현되어 있는 API Path를 개선하였습니다.

Path 변경 사항

기능 HTTP 메서드 URI 경로
참가자 로그인 POST /api/v1/meetings/{uuid}/login
약속 정보 생성 POST /api/v1/meetings
약속 정보 조회 GET /api/v1/meetings/{uuid}
약속 공유 조회 GET /api/v1/meetings/{uuid}/sharing
일정 생성 POST /api/v1/meetings/{uuid}/schedules
약속의 모든 일정 조회 GET /api/v1/meetings/{uuid}/schedules
약속의 특정 회원 일정 조회 GET /api/v1/meetings/{uuid}/schedules?attendeeName=
약속의 내 일정 조회 GET /api/v1/meetings/{uuid}/attendees/me/schedules

특이 사항

  • 자원 식별 혼동을 줄이기 위해 모든 자원에 대해 복수형 명사를 사용하였습니다.
  • uuid를 사용하는 path에 일괄적으로 meetings를 명시하여 계층 구분을 명확히 하였습니다.
  • 약속의 내 일정 조회 기능의 경우 현재 인증된 사용자 정보로 일정에 접근하는 의미를 명확히 하도록 수정하였습니다.
    • 자원의 계층을 분리하여 URI 일관성을 유지하였습니다.
    • me 키워드를 통해 현재 일정 자원에 접근하는 참가자가 인증된 참가자라는 의미를 포함합니다.

리뷰 요구사항 (선택)

@seokmyungham seokmyungham added 🐈 프론트엔드 프론트엔드 관련 이슈에요 :) 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :) ♻️ 리팩터링 코드를 깎아요 :) labels Jul 29, 2024
@seokmyungham seokmyungham added this to the 3차 데모데이 milestone Jul 29, 2024
@seokmyungham seokmyungham self-assigned this Jul 29, 2024
Copy link

github-actions bot commented Jul 29, 2024

Test Results

47 tests   47 ✅  4s ⏱️
14 suites   0 💤
14 files     0 ❌

Results for commit 202b514.

♻️ This comment has been updated with latest results.

Copy link
Contributor

@hwinkr hwinkr 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

@hw0603 hw0603 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
Contributor

@Yoonkyoungme Yoonkyoungme left a comment

Choose a reason for hiding this comment

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

🎺🎷재즈🎺🎷 고생하셨어요👍👍👏

약속의 내 일정 조회 GET /api/v1/meetings/{uuid}/attendees/me/schedules

mecurrent도 가볍게 제안드려 봅니다!

약속의 내 일정 조회 GET /api/v1/meetings/{uuid}/attendees/current/schedules

Copy link
Contributor

@ikjo39 ikjo39 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
Contributor

@seunghye218 seunghye218 left a comment

Choose a reason for hiding this comment

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

모호했던 API URL이 일관되고 명확해졌네요😄

Copy link
Contributor

@ehBeak ehBeak 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
Contributor

@Largopie Largopie left a comment

Choose a reason for hiding this comment

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

REST API 관련된 레퍼런스도 함께 이슈에 첨부해주어서 이해하는데 많은 도움이 되었어요! 고생하셨습니다 :) 훨씬 직관적인 것 같네요 👍

@seokmyungham seokmyungham merged commit 8d9cbcc into develop Jul 30, 2024
10 checks passed
@ikjo39 ikjo39 deleted the refactor/104-api-path branch July 31, 2024 01:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
♻️ 리팩터링 코드를 깎아요 :) 🐈 프론트엔드 프론트엔드 관련 이슈에요 :) 🐈‍⬛ 백엔드 백엔드 관련 이슈에요 :)
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

[ALL] RESTful API 디자인 원칙에 기반하여 기존 API path를 수정해요 :)
8 participants