feat: 히스토리 api ts rest 적용 #599

nyj001012 · 2023-07-29T08:28:57Z

개요

히스토리 API에 ts-rest 적용하여 리팩토링

작업 사항

router
controller
service
error

비고

Impl.ts 파일 분리 #600
book Id 안 쓰는거 같은데 없애면 안되나
api 나누기 (유저 권한에 따라 등) - who로 구별하기 번거로움

스크린샷 (optional)

scarf005 · 2023-07-31T05:17:29Z

https://github.com/orgs/jiphyeonjeon-42/discussions/7#discussioncomment-6581705

nyj001012 · 2023-08-02T08:01:49Z

@scarf005

{
  "issues": [
    {
      "code": "invalid_type",
      "expected": "number",
      "received": "string",
      "path": [
        "page"
      ],
      "message": "Expected number, received string"
    },
    {
      "code": "invalid_type",
      "expected": "number",
      "received": "string",
      "path": [
        "limit"
      ],
      "message": "Expected number, received string"
    }
  ],
  "name": "ZodError"
}

page와 limit가 string으로 들어가요! help...

histories 라우터에 추가

histories controller 추가

impl 추가

histories 전용 type 파일 추가

histories service 추가

권한 없음 에러 추가

권한 없음 에러 오브젝트 추가

meta 스키마 추가

eslint comma 추가

contract에 histories 추가

contract에 histories 명세 추가

contract에 histories에서 사용하는 GET 메서드들의 query string 스키마 추가

contract에 공용으로 사용할 권한 없음 에러 스키마 추가

- 전체 대출 기록 조회하는 controller 추가 - 나의 대출 기록 조회하는 controller 추가

- 전체 대출 기록 조회하는 service 추가 - 나의 대출 기록 조회하는 service 추가

histories 조회 시의 query string schema 구조 변경

VHistories 가져올 때 중괄호 이용

histories를 조회하여 200 스테이터스 일 때 반환하는 스키마 추가

callsign으로도 검색할 수 있도록 조건 추가 및 query undefined 체크 추가

meta 스키마 공용 스키마로 추가

id 타입을 string -> number로 변경

z.number()...를 positiveInt로 변경

서비스 함수 반환 타입 오브젝트 형식으로 변경

getMyHistories 반환값 데이터베이스 조회 결과로 변경

서비스 함수들 반환 타입 변경

서비스 파일과 서비스 구현 파일, 컨트롤러 파일과 컨트롤러 구현 파일 분리

scarf005 · 2023-08-07T07:41:26Z

contracts/src/histories/schema.ts

+      createdAt: z.string(),
+      returnedAt: z.string(),
+      updatedAt: z.date(),


VHistories의 createdAt 필드의 타입은 Date이나, createdAt의 타입이 string이어서 타입 오류가 나고 있습니다. 하지만 typeorm 쿼리에서 날짜를 문자열 형식으로 반환하기 때문에, 이와 같은 경우에는 coerce를 사용하는 것을 추천드립니다.

colinhacks/zod#879

scarf005 · 2023-08-07T07:45:08Z

backend/src/v2/histories/controller/controller.ts

+  return match(result)
+    .with(P.instanceOf(UnauthorizedError), () => unauthorized)
+    .otherwise(() => ({
+      status: 200,
+      body: result,
+    } as const));


반환값이 UnauthorizedError | { items: [...], meta: { ... } } 타입이기 때문에, UnauthorizedError를 예상하지 못한 ts-rest에서 오류를 내고 있습니다. 다음을 참고하시면 타입을 좁힐 때 도움이 될 것 같습니다.

https://github.com/gvergnaud/ts-pattern#withpattern-handler
https://github.com/gvergnaud/ts-pattern#exhaustive-otherwise-and-run

scarf005

LGTM! 시간을 끌어 죄송합니다...
마지막에 #663 을 위한 타입 수정도 조금 추가했습니다

nyj001012 added the refactoring label Jul 29, 2023

nyj001012 assigned nyj001012 and Yonaim Jul 29, 2023

nyj001012 linked an issue Jul 29, 2023 that may be closed by this pull request

☂️ v2 리팩토링 - histories API #598

Closed

nyj001012 temporarily deployed to development July 29, 2023 08:29 — with GitHub Actions Inactive

scarf005 self-requested a review July 29, 2023 08:48

scarf005 self-assigned this Jul 31, 2023

scarf005 mentioned this pull request Jul 31, 2023

☂️ v2 리팩토링 - histories API #598

Closed

nyj001012 had a problem deploying to development August 1, 2023 08:14 — with GitHub Actions Failure

nyj001012 linked an issue Aug 1, 2023 that may be closed by this pull request

관리자 및 사용자용 API 경로 정리 #615

Closed

nyj001012 had a problem deploying to development August 2, 2023 08:20 — with GitHub Actions Failure

scarf005 had a problem deploying to development August 2, 2023 08:26 — with GitHub Actions Failure

nyj001012 added 16 commits August 3, 2023 22:09

feat: histories 라우터에 추가

6bc0872

histories 라우터에 추가

feat: histories controller 추가

642c3ae

histories controller 추가

feat: impl 추가

0123076

impl 추가

feat: histories 전용 type 파일 추가

ae75767

histories 전용 type 파일 추가

feat: histories service 추가

a03eb43

histories service 추가

feat: 권한 없음 에러 추가

88a9052

권한 없음 에러 추가

feat: 권한 없음 에러 오브젝트 추가

3153397

권한 없음 에러 오브젝트 추가

feat: meta 스키마 추가

99b5742

meta 스키마 추가

style: eslint comma 추가

e4d6bae

eslint comma 추가

feat: contract에 histories 추가

b29e417

contract에 histories 추가

feat: contract에 histories 명세 추가

4457518

contract에 histories 명세 추가

feat: contract에 histories에서 사용하는 query string 스키마 추가

399233b

contract에 histories에서 사용하는 GET 메서드들의 query string 스키마 추가

feat: contract에 공용으로 사용할 권한 없음 에러 스키마 추가

7c4ad3c

contract에 공용으로 사용할 권한 없음 에러 스키마 추가

feat(histories): histories controller 추가

ba76a8d

- 전체 대출 기록 조회하는 controller 추가 - 나의 대출 기록 조회하는 controller 추가

feat(histories): histories service 추가

37080d6

- 전체 대출 기록 조회하는 service 추가 - 나의 대출 기록 조회하는 service 추가

feat(histories): histories 조회 시의 query string schema 변경

3df6b8d

histories 조회 시의 query string schema 구조 변경

nyj001012 had a problem deploying to development August 7, 2023 02:32 — with GitHub Actions Failure

style(histories): import 형식 변경

1d956ef

VHistories 가져올 때 중괄호 이용

nyj001012 had a problem deploying to development August 7, 2023 02:34 — with GitHub Actions Failure

nyj001012 added 4 commits August 7, 2023 11:34

feat(histories): histories 조회 결과 200 시, 스키마 추가

ebc2df1

histories를 조회하여 200 스테이터스 일 때 반환하는 스키마 추가

feat(histories): 검색 조건 callsign 추가

2876193

callsign으로도 검색할 수 있도록 조건 추가 및 query undefined 체크 추가

feat(histories): meta 스키마 추가

a90eb0c

meta 스키마 공용 스키마로 추가

feat(histories): id 타입 변경

b81689c

id 타입을 string -> number로 변경

nyj001012 had a problem deploying to development August 7, 2023 02:36 — with GitHub Actions Failure

feat(histories): meta 스키마 positiveInt로 변경

8356875

z.number()...를 positiveInt로 변경

nyj001012 had a problem deploying to development August 7, 2023 02:44 — with GitHub Actions Failure

feat(histories): 서비스 함수 반환 타입 변경

4636b8a

서비스 함수 반환 타입 오브젝트 형식으로 변경

nyj001012 had a problem deploying to development August 7, 2023 03:03 — with GitHub Actions Failure

nyj001012 added 2 commits August 7, 2023 12:11

feat(histories): getMyHistories 반환값 변경

a91fea2

getMyHistories 반환값 데이터베이스 조회 결과로 변경

feat(histories): 서비스 함수 반환 타입 변경

9c5b560

서비스 함수들 반환 타입 변경

nyj001012 had a problem deploying to development August 7, 2023 03:12 — with GitHub Actions Failure

chore(histories): 서비스 파일 분리 및 컨트롤러 파일 분리

c2ed994

서비스 파일과 서비스 구현 파일, 컨트롤러 파일과 컨트롤러 구현 파일 분리

nyj001012 had a problem deploying to development August 7, 2023 03:25 — with GitHub Actions Failure

scarf005 requested changes Aug 7, 2023

View reviewed changes

scarf005 self-requested a review August 13, 2023 03:20

Merge branch 'develop' into 598-히스토리-api-ts-rest-적용

840747e

scarf005 marked this pull request as ready for review August 13, 2023 03:49

scarf005 added 2 commits August 13, 2023 12:57

fix: 스키마에서 date와 string 모두 허용

eec4ade

fix: 패턴 매칭에서 타입이 좁혀지지 않던 문제

2aa991f

scarf005 had a problem deploying to development August 13, 2023 03:59 — with GitHub Actions Failure

scarf005 added 2 commits August 13, 2023 12:59

fix: users 경로 숨김

d32ded4

fix: stocks 타입 오류 수정

12c264b

scarf005 temporarily deployed to development August 13, 2023 04:03 — with GitHub Actions Inactive

scarf005 approved these changes Aug 13, 2023

View reviewed changes

scarf005 merged commit 27b3d11 into develop Aug 13, 2023
2 checks passed

scarf005 deleted the 598-히스토리-api-ts-rest-적용 branch August 13, 2023 04:06

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

feat: 히스토리 api ts rest 적용 #599

feat: 히스토리 api ts rest 적용 #599

nyj001012 commented Jul 29, 2023 •

edited by scarf005

Loading

scarf005 commented Jul 31, 2023

nyj001012 commented Aug 2, 2023

scarf005 Aug 7, 2023

scarf005 Aug 7, 2023 •

edited

Loading

scarf005 left a comment

feat: 히스토리 api ts rest 적용 #599

feat: 히스토리 api ts rest 적용 #599

Conversation

nyj001012 commented Jul 29, 2023 • edited by scarf005 Loading

개요

작업 사항

비고

스크린샷 (optional)

scarf005 commented Jul 31, 2023

nyj001012 commented Aug 2, 2023

scarf005 Aug 7, 2023

Choose a reason for hiding this comment

scarf005 Aug 7, 2023 • edited Loading

Choose a reason for hiding this comment

scarf005 left a comment

Choose a reason for hiding this comment

nyj001012 commented Jul 29, 2023 •

edited by scarf005

Loading

scarf005 Aug 7, 2023 •

edited

Loading