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.

Sign up for GitHub

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

Jump to bottom

refactor: lendings에 v2 kysely 적용 #792

Draft
wants to merge 39 commits into
base: develop
Choose a base branch
from
Draft

refactor: lendings에 v2 kysely 적용 #792

wants to merge 39 commits into from

Conversation

nyj001012
Copy link
Contributor

개요

lendings의 /api/v2/mypage/lendings, /api/v2/lendings (구 histories)에 kysely 적용

작업 사항

  • kysely codegen으로 만든 DB 스키마와 contracts에 정의된 response 스키마 동기화
  • TypeORM 대신 kysely 사용

scarf005 and others added 21 commits September 7, 2023 18:19
@scarf005
ci: 타입오류 메시지 상대경로 명시적으로 해소 (#643)
fa9559c 
참고: actions/runner#659
@JeongJiHwan @scarf005
feat: v2 book api (#746)
93fc7b4 
* 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>
@scarf005
style: prettier 적용 (#761)
8b7c07f 
* build: prettier 설정

* style: prettier 적용

---------

Co-authored-by: nocontribute <>
@weg901127 @kylee001
[fix] backend dockerfile error (#764)
c88ca8d 
Co-authored-by: kylee <kylee@fitfuns.com>
@scarf005
fix: positiveInt -> nonNegativeInt (#766)
1565441
@scarf005 @Jiwon-Woo
@honeyl3ee @JeongJiHwan
refactor: v2 라우트 정리 적용 (#771)
6cbb823 
* 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>
@jimin52
feat: add mydata service
1ac5b23 
토큰에서 id 정보 찾아서 유저 정보 반환하는 controller
@jimin52
feat: 유저 search 할 때 id 가 undefined 인 경우 핸들링
7094331
@jimin52
feat: add swagger && /me endpoint && apply authValidate
1c8b49b
@jimin52
Merge branch 'develop' into 778-auth-관련-api-무조건-200-리턴-버그
71c1216
@jimin52
fix: searchUsersById 타입을 이전과 같이 리턴하도록 변경
5754bef 
searchUsersById 서비스 함수의 종속성이 생각보다 많음.controller 에서 items 의 length 를 확인하도록
@jimin52
Merge branch '778-auth-관련-api-무조건-200-리턴-버그' of https://github.com/ji…
7fbfa10 
…phyeonjeon-42/backend into 778-auth-관련-api-무조건-200-리턴-버그
@jimin52
fix: add librarian validate in search endpoint
1449028
@jimin52
feat: 로그인한 유저만 본인 정보를 찾을 수 있도록 middleware 에서 권한 체크
cda9990
@jimin52
chore: console.log 제거
7b7ad82
@jimin52
Merge pull request #779 from jiphyeonjeon-42/778-auth-관련-api-무조건-200-…
c5bd7ae 
…리턴-버그

fix: auth 관련 api가 무조건 200 리턴 버그
@nyj001012
User API 경로 정리 (#777)
cf970dd 
* refactor: 400번대 에러 반환 제거

* refactor: overDueDay 반환 값에서 제거
@nyj001012
fix(cursus): Access-Control-Allow-Origin 설정 (#790)
d7bca34
@nyj001012
style(v2): 불필요한 import 삭제
d8c7972
@nyj001012
feat(contracts): kysely generator에서 만들어준 스키마와 타입 동일하게 함
3699ae0
@nyj001012
refactor(v2): 마이페이지 대출기록 함수에 kysely 적용
0b068bb
@nyj001012 nyj001012 added the enhancement New feature or request label Oct 28, 2023
@nyj001012 nyj001012 self-assigned this Oct 28, 2023
@nyj001012 nyj001012 linked an issue Oct 28, 2023 that may be closed by this pull request
V2 kysely 시범 사용 #783
Open
@nyj001012
chore: dependencies 업데이트 (#796)
53da6fd 
* 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 동기화
@nyj001012 nyj001012 force-pushed the 783-v2-kysely-시범-사용 branch from db53668 to daec465 Compare November 6, 2023 07:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@nyj001012 nyj001012

Labels
enhancement New feature or request
Projects
집현전 v2 리팩터링
Status: 작업중
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

V2 kysely 시범 사용
5 participants
@nyj001012 @scarf005 @JeongJiHwan @weg901127 @jimin52