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

feat: 793 reservation 라우트 경로 정리 #795

Open
wants to merge 4 commits into
base: develop
Choose a base branch
from

Conversation

JeongJiHwan
Copy link
Contributor

@JeongJiHwan JeongJiHwan commented Oct 30, 2023

개요

  • Reservation 라우트 경로 정리 #793 정리 후 reservations contracts 작성

    v1 : [GET] api/reservations/
    본인이 예약한 책을 확인할 수 있는 api로 확인되나 마이페이지에서는 user api에서 예약한 도서를 확인하고 있습니다.
    [GET] api/mypage/reservations 로 변경하는게 맞을까요?

이 부분도 일단 contract에 추가해 두었습니다. 피드백이 있다면 부탁드립니다.

[get] reservations response에서 확인되는 status 값은
0. 예약중

  1. 대출완료
  2. 예약취소
  3. 예약기한만료
    로 확인을 했습니다.

[get] reservations request query 에서의 filter 값은

  • pending : 예약 0순위
  • waiting : 예약 후순위
  • expired : 종료된 예약
    으로 확인했습니다.

개인 예약 조회, 예약 조회, 도서별 예약 조회 부분 작성완료, 예약 생성, 예약 취소 부분 작성 필요
@JeongJiHwan JeongJiHwan linked an issue Oct 30, 2023 that may be closed by this pull request
@JeongJiHwan JeongJiHwan changed the title 793 reservation 라우트 경로 정리 [feat] 793 reservation 라우트 경로 정리 Nov 6, 2023
@JeongJiHwan JeongJiHwan changed the title [feat] 793 reservation 라우트 경로 정리 feat: 793 reservation 라우트 경로 정리 Nov 6, 2023
예약 취소 api 에 cancel 을 명시적으로 표현하는 쪽이 더 낫다고 판단하여 수정하였습니다.
Copy link
Member

@jimin52 jimin52 left a comment

Choose a reason for hiding this comment

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

잘 정리된 것 같습니다.

export const reservationsContract = c.router({
getMine: {
method: 'GET',
path: '/mypage/reservations',
Copy link
Member

Choose a reason for hiding this comment

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

/reservations/me
/me/reservations
로 해도 좋을 것 같습니다.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: 할 일
Development

Successfully merging this pull request may close these issues.

Reservation 라우트 경로 정리
2 participants