refactor: Must return HTTP Status OK(200) #61
Merged
Conversation
This file contains 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
# Conflicts: # src/domain/auth/controller/auth.controller.ts
son-daehyeon
added a commit
that referenced
this pull request
Aug 17, 2024
* auth의 swagger 문서 수정 (#38) * style(auth): 데코레이터 이름 수정 * docs(auth): getMyInfo에 ErrorResponse 추가 * Features/s3 (#39) * chore(s3): 패키지 추가 * chore(s3): multer 설정 * style: import 오더 수정 * Features/member (#40) * chore(test): 삭제 모듈 완성 후 다시 작성 * style(auth): 데코레이터 이름 수정 * docs(auth): getMyInfo에 ErrorResponse 추가 * chore(s3): 패키지 추가 * chore(s3): multer 설정 * feat(member): DTO 생성 * feat(member): Controller 형태 생성 * chore(auth): 권한 부족 오류 분리 * feat(member): 서비스 구현 * chore(util): 전역 생성자 분리 * feat(member): 아바타 변경 시 기존 아바타 삭제 * feat(member): 아바타 변경 시 기존 아바타 삭제 * chore: 주석 삭제 * chore: EmailTemplate을 전역 유틸로 * feat: 이메일 템플릿 수정 * style: userId to memberId * test(auth): 이메일 모듈 모킹 수정 * style: 단어 통일 (거절 -> 거부) * style: 단어 통일 (유저 -> 멤버) * chore: memberId example 수정 * chore: DTO validator 수정 * test: DTO validator 테스트 추가 * fix: Auth Decorator 수정 * chore(member): dto swagger example 수정 * chore: Auth 관련 커스텀 exception 추가 * chore: 사용하지 않는 ObjectId 삭제 * refactor: mock util을 공통 util로 * fix: config/config.template.yaml * fix: 기존 아바타 삭제 로직 수정 * Refactor/daehyeon (#41) * refactor(logger): purge LoggerMiddleware Nginx logger 사용 * refactor(mail): 메일 전송 기다리지 않기 * refactor(validation): 모든 검증자 커스텀 * refactor: import order * fix: set global api prefix * refactor: purge swagger theme * refactor: 코드 최적화 (#42) * chore(member): 단어 통일 (아바타 -> 프로필 사진) * docs(README): docker 명령어 수정 * fix: swagger url 조정 * fix: Common response 수정 * feat(s3): multerS3 대신 직접 aws로 업로드 * test(member): Validation 테스트 수정 * feat: 유틸 및 특정 부분 로깅 * chore: email from 변경 * refactor: 코드 최적화 및 버그 수정 (#43) * chore: 단어 통일 (refuse to reject) * chore: api schema 변경 * chore: 승인 여부 필드를 분리 * fix(auth-test): #0bbadb 케이스 수정 * chore: allowSyntheticDefaultImports * feat: mongoose versionKey 삭제 및 timestamps 추가 * refactor(auth): myInfo service 분리 * refactor(member): 스키마 필드 추가 * refactor(member): 멤버 조회 부분 코드 수정 * refactor: schema _id to id * refactor(revert): allowSyntheticDefaultImports * style: 폴더 구조 수정 * feat: 학교 메일만 가입 가능하도록 * fix: MemberRepository 수정 * fix(auth-vali): test 수정 * refactor(struct): 폴더 구조 변경 (#44) * refactor: DynamicModule화 (#45) * refactor(redis): DynamicModule화 * refactor(s3): DynamicModule화 * refactor(test): DynamicModule화 * Refactor: auth test (#47) * test(auth): mock spy * fix(member): Omit시 구조 분해 사용 X * test(auth): 내 정보 조회 서비스 추가 * Delete spec/auth/auth.integrated.spec.ts * Refactor: member test (#46) * test: Mocking S3Service * test: Mocking Member Module * test(fix): MemberRepository 수정 * style(member): import order * test: MemberService 테스트 * test(s3): mock object의 return값 지정 * test(member): mock spy * fix(member): Omit시 구조 분해 사용 X * fix(mock): 잘못된 mock 수정 * test: MemberAdminService 테스트 * test(auth): Purge 통합 테스트 곧 E2E 테스트 사용 예정 * feat: Scheduler 구현 (#48) * feat(s3): 모든 파일 조회 * chore(scheduler): @nestjs/scheduler 라이브러리 추가 * chore(scheduler): ScheduleModule 등록 * feat(scheduler): 사용하지 않는 아바타 삭제 * refactor(s3): S3Service 관점으로 key 사용 * test(s3): S3Service 변화 적용 * refactor: Typescript 엄격한 옵션 on (#49) * refactor: validation 수정 * refactor: 각종 테스트 리팩토링 (#50) * test(mock): MemberRepository 최신화 * test(auth-service): login#MemberNotFoundException 추가 * test(auth-validation): 테스트 이름 영어로 * fix(member): DTO Optional이었던 것 수정 * refactor(test-member): 빈 멤버 만드는 팩토리 * refactor(test): 리팩토링 * docs(swagger): API Docs 최신화 (#51) * refactor: 컨트롤러 할 일 서비스에 더 위임 (#52) * style(member): 파일 네이밍 (AvatarFilter to avatar-filter) * refactor(auth): Controller 일 Service 로 모두 위임 * refactor(member): Controller 일 Service 로 모두 위임 * test: Controller 일 Service 로 모두 위임 * refactor: 자잘한 것 리팩토링 (#53) * refactor(mail): 템플릿 class화 * chore: format 및 lint 스크립트에 spec폴더도 추가 * refactor: 중복 promise 리턴 삭제 * feat: Swagger 오류 내용을 포함한 ApiException 인터페이스 추가 * refactor: StatusCode를 숫자 말고 HttpStatus Enum으로 * feat(member): 특정 필드 제거해주는 유틸 생성 * refactor: Role 세분화 및 권한 위계 관리 (#54) * chore: function to arrow function * feat(role): Role 세분화 및 권한 위계 관리 * fix(test); MemberValidation 수정 * feat(role): Role 세분화 및 권한 위계 관리 * refactor(swagger): 같은 오류 들어오면 하나만 출력 * refactor: 이벤트 기반 설계 (#55) * refactor(event): 이벤트 기반 설계 * refactor(event): 이벤트 기반 설계 * fix(test): 몇몇 수정사항 적용 * chore(mock): complexity 경고 억제 * refactor: import alias (#56) * feat(alias): import alias 적용 * refactor(alias): Alias 설정 * chore: config 파일 정리 (#57) * refactor: code style (#58) * chore(folder): common의 폴더 구조 세분화 * refactor: 코드 스타일 수정 * fix(member): 버그 수정 * refactor(member): 승인/미승인 멤버 find를 쿼리로 변경 * Feat: JWT Refresh Token (#60) * chore(util): ms 패키지 추가 * feat: DTO 수정 * feat: Exception 수정 * test: Mocking 및 테스트 추가 * refactor: jwt expires를 호출 당시에 쓰도록 * feat: 리프레시 토큰 구현 * refactor: Must return HTTP Status OK(200) (#61) * chore(util): ms 패키지 추가 * feat: DTO 수정 * feat: Exception 수정 * test: Mocking 및 테스트 추가 * refactor: jwt expires를 호출 당시에 쓰도록 * feat: 리프레시 토큰 구현 * refactor: Must be return HTTP Status OK(200)
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
작업 내용
기타