[오연진] sprint5#176
Open
yonjinoh wants to merge 55 commits intocodeit-bootcamp-nodejs:오연진from
Hidden character warning
The head ref may contain hidden characters: "\uc624\uc5f0\uc9c4-sprint4"
Open
Conversation
불필요한 주석 제거 및 User 모델 추가
회원가입 API 구현 토큰 기반 로그인 기능 구현
User 모델과 상품 기능 1:N 관계 설정 쿠키 accessToken 누락 수정 쿠키에서 accessToken 추출 미들웨어 추가 상품 기능 인가 추가
해싱 관련 미들웨어를 만들었는데, 비밀번호 수정에서 해당 미들웨어 사용가능한지 확인해보기 지금은 service단에만 구현함
zod로 구현 이후에 기존 유효성 검증 미들웨어도 zod로 바꾸기
게시글 상품 상세 조회 시 에러 메시지 수정도 진행
validate.js 에러 메시지 오류 수정 불필요한 import 제거 토큰 생성 함수 분리
불필요해진 주석 제거 및 회원가입 로직 리팩토링 기존 미들웨어로 관리되던 해싱 관련 파일을 libs로 이동
초급 프로젝트에서 진행한 방식으로 이미지 클라우드 사용
tsconfig.json 파일을 생성하고, 필요한 옵션을 설정 필요한 npm script를 설정 ts-node nodemon을 사용해 .ts 코드가 변경될 때마다 서버가 다시 실행되는 npm script
undefined 타입 오류 수정 main 주석 처리 제거 및 테스트 진행 errorHandler에 error any type으로 처리하던거 customError로 수정
types 폴더 아래에 dtos 폴더 추가 article dto 추가 validate.ts validators 폴더 밖으로 분리 (validators 폴더 제거 예정) - 관련 import 구문 수정 controller, service, router 수정
config/env와 libs/constant로 중복 관리되던 환경변수 상수들 libs/constant로 통일
생성: 201 조회/수정: 200 삭제: 204
기존 zod와 type으로 분리해 관리하던 데이터 타입을 class-validator를 사용해 하나로 관리
like type 파라미터 union타입 사용 req.user 값에 모든 데이터를 넣지 않고 id, email, nickname만 넣기 중복된 데코레이터 제거 상태코드 204 -> 200 strict equal로 변경
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
1 participant
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.
멘토에게