feat: 793 reservation 라우트 경로 정리#795
Open
JeongJiHwan wants to merge 23 commits intodevelopfrom
Hidden character warning
The head ref may contain hidden characters: "793-reservation-\ub77c\uc6b0\ud2b8-\uacbd\ub85c-\uc815\ub9ac"
Open
Conversation
* refactor: v1 api와 다른 부분 수정 * refactor: books API schema default sort value 추가 및 callSign 오타 수정 * refactor: status: zod.enum -> zod.nativeEnum 변경 z.enum으로 했을때 number 반환값에 대한 처리가 되지 않아 enumStatus 생성 후 nativeEnum으로 변경하였습니다. * feat: books//:id v2 구현 * feat: books/search v2 구현 * feat: book/update v2 구현 * feat: books/info/:id v2 구현 * refactor: BookNotFoundError import 오타 수정 * feat: books/info/sorted v2 구현 * feat: books/info/tag v2 구현 book_info.id 로 distinct가 되지 않는 오류가 있어요 * refactor: 의미에 맞는 변수명으로 수정 및 코드 간략화 Co-authored-by: scarf <greenscarf005@gmail.com> * refactor: book_info 중복 select 부분 처리 Co-authored-by: scarf <greenscarf005@gmail.com> * feat: books/donator v2 구현 v1 구현 시 얘기했었던 기부자가 유저가 아니더라도 수정될 수 있도록 수정 & email은 더이상 관리하지 않기 때문에 변수명에서 제거 * refactor: 대출 가능 여부 boolean 반환값으로 수정 kysely 사용중 select가 없을 경우 sql syntax 에러로 해당 부분 수정 및 boolean 값 반환되도록 수정 * feat: [get] books/create v2 구현 axios.get 동작 중 발생하는 에러(catch 영역)에 대한 처리를 어떻게 해야할지 모르겠어요 * fix: 임시로 타입 오류 무시 --------- Co-authored-by: scarf <greenscarf005@gmail.com>
* build: prettier 설정 * style: prettier 적용 --------- Co-authored-by: nocontribute <>
Co-authored-by: kylee <kylee@fitfuns.com>
* refactor: `/stock` 제거 #767 (comment) * refactor: contracts에서 500번대 오류 제거 백엔드에서 복구 불가능한 오류일 시 반환하기 때문에, 프론트엔드에서 따로 처리하는 것이 좋을 것 같습니다. * refactor: `/history` -> `/lendings` #767 (comment) Co-authored-by: jwoo <74581396+Jiwon-Woo@users.noreply.github.com> * refactor: `/users ` 정리 #767 (comment) Co-authored-by: honeyl3ee <ddanhopark@gmail.com> * refactor: `/tag` 서비스 임시 제거 고도화를 하기 위해서는 내부 구현을 바꾸어야 하는 문제가 있어 우선순위를 낮추었습니다. 주석처리를 할까 고민했으나 제거 이전 커밋(1565441)으로 체크아웃시 전체 코드를 확인 가능하기 때문에 복잡도 감소를 위해 코드를 제거하였습니다. * refactor: `/books` 경로 정리 #767 (comment) #767 (comment) Co-authored-by: Jeong Jihwan <47599349+JeongJiHwan@users.noreply.github.com> Co-authored-by: jwoo <74581396+Jiwon-Woo@users.noreply.github.com> * feat: swagger에서 1줄 요약 표시 --------- Co-authored-by: jwoo <74581396+Jiwon-Woo@users.noreply.github.com> Co-authored-by: honeyl3ee <ddanhopark@gmail.com> Co-authored-by: Jeong Jihwan <47599349+JeongJiHwan@users.noreply.github.com>
토큰에서 id 정보 찾아서 유저 정보 반환하는 controller
searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록
…phyeonjeon-42/backend into 778-auth-관련-api-무조건-200-리턴-버그
…리턴-버그 fix: auth 관련 api가 무조건 200 리턴 버그
* refactor: 400번대 에러 반환 제거 * refactor: overDueDay 반환 값에서 제거
개인 예약 조회, 예약 조회, 도서별 예약 조회 부분 작성완료, 예약 생성, 예약 취소 부분 작성 필요
JeongJiHwan
had a problem deploying
to
development
GitHub Actions
Failure
* chore(deps): contracts의 pnpm-lock 업데이트 * chore(deps): @mapbox/npm-pre-gyp 설치 * chore(deps): npm-pre-gyp 설치 * Revert "chore(deps): npm-pre-gyp 설치" This reverts commit 8922c38 * chore(deps): package.json과 pnpm-lock.yaml 동기화
…-reservation-라우트-경로-정리
JeongJiHwan
temporarily deployed
to
development
GitHub Actions
Inactive
JeongJiHwan
changed the title
JeongJiHwan
changed the title
예약 취소 api 에 cancel 을 명시적으로 표현하는 쪽이 더 낫다고 판단하여 수정하였습니다.
JeongJiHwan
temporarily deployed
to
development
GitHub Actions
Inactive
jimin52
reviewed
Dec 3, 2023
| export const reservationsContract = c.router({ | ||
| getMine: { | ||
| method: 'GET', | ||
| path: '/mypage/reservations', |
Member
There was a problem hiding this comment.
/reservations/me
/me/reservations
로 해도 좋을 것 같습니다.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
5 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
개요
Reservation 라우트 경로 정리 #793 정리 후 reservations contracts 작성
v1 : [GET] api/reservations/
본인이 예약한 책을 확인할 수 있는 api로 확인되나 마이페이지에서는 user api에서 예약한 도서를 확인하고 있습니다.
[GET] api/mypage/reservations 로 변경하는게 맞을까요?
이 부분도 일단 contract에 추가해 두었습니다. 피드백이 있다면 부탁드립니다.
[get] reservations response에서 확인되는 status 값은
0. 예약중
로 확인을 했습니다.
[get] reservations request query 에서의 filter 값은
으로 확인했습니다.