[feat] Add mongo id form valid checker #124

geeneve · 2022-03-05T08:33:11Z

🌈 PR 요약 / Linked Issue

Request로 받은 id들이 mongodb id 형식인지 확인하는 코드 추가
close [feat] Check mongo ID valid #123

📌 변경 사항

야무진 라이브러리 import { isValidObjectId } from "mongoose";를 사용해 id 형식 확인하는 코드를 추가했습니다.
만약 유효하지 않을 때 보내야 하는 response를 추가했습니다.
유효하지 않을 때 WRONG_REQUEST_VALUE를 쓰기로 해, 기존에 WRONG_REQUEST_VALUE를 사용하던 부분은 DB_NOT_FOUND로 수정했습니다.

✅ PR check list

1. 제목 양식 준수 확인

[feat] PR title

2. 테스트 코드 작동 스크린샷

npm run mocha

✅ After merge check list

1. 릴리즈 노트 변경

릴리즈 노트 To Do에서 Done으로 옮기기

2. 브랜치 삭제

holmir97 · 2022-03-05T09:10:05Z

src/service/review.ts

@@ -3,6 +3,7 @@ import mongoose from "mongoose";
 // library
 import constant from "../library/constant";
 import { keysToSnake, keysToCamel } from "../library/convertSnakeToCamel";
+import { isValidObjectId } from "mongoose";


feat: add mongo id form valid checker

4d926cd

geeneve added 🍀 feature 새로운 기능 추가 3️⃣ priority: low 우선순위 낮음 labels Mar 5, 2022

geeneve added this to the v2.0.1 milestone Mar 5, 2022

geeneve requested review from seohyun-106 and holmir97 March 5, 2022 08:33

geeneve self-assigned this Mar 5, 2022

holmir97 approved these changes Mar 5, 2022

View reviewed changes

holmir97 merged commit 1044a74 into develop Mar 5, 2022

holmir97 deleted the feat/mongoid-valid-check branch March 5, 2022 09:11

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[feat] Add mongo id form valid checker #124

[feat] Add mongo id form valid checker #124

geeneve commented Mar 5, 2022

holmir97 Mar 5, 2022

[feat] Add mongo id form valid checker #124

[feat] Add mongo id form valid checker #124

Conversation

geeneve commented Mar 5, 2022

🌈 PR 요약 / Linked Issue

📌 변경 사항

✅ PR check list

1. 제목 양식 준수 확인

2. 테스트 코드 작동 스크린샷

✅ After merge check list

1. 릴리즈 노트 변경

2. 브랜치 삭제

holmir97 Mar 5, 2022

Choose a reason for hiding this comment