Conversation
기존 단일 레벨이었던 태그에 Depth를 추가했습니다. * 2-Depth 제한은 ApplicationLayer에 추가할 예정입니다.
JwtStrategy의 인스턴스화에서 ConfigService가 undefined로 주입되어 발생하던 문제를 수정했습니다.
JwtStrategy와 User 데코레이터의 반환타입을 수정했습니다.
- API 응답을 명확화 하기 위해 컨트롤러와 DTO에 스웨거 문서 작성 - 추후 펀딩 진행률에 캐싱을 적용할 예정
feat/me: 내 페이지 및 자잘한 오류 수정
기존 단일 Course 모델이 처리하지 못하는 두 가지 주요 비즈니스 로직을 분리하기 위해 재설계를 진행했습니다. - 기존 Course를 강의 소개/커리큘럼 등 공통 정보를 담도록 하고, 이외 고유 속성은 Track과 Crew모델로 분리했습니다. (데이터베이스 레벨에서는 완벽하게 보장되지는 않습니다) - 기존 Enrollment도 TrackEnrollment와 CrewEnrollment로 분리하였습니다.
- 데이터베이스 쿼리를 담당하는 CourseRepository 추가 - PrismaValidator를 사용해 조회 결과에 대한 타입 보장 - API 요청/응답의 규격 DTO 추가
- CourseRepository를 이용해 데이터를 조회 후 DTO형태로 가공하는 비즈니스 로직 수행 - private 헬퍼 메서드로 일부 작업들을 분리함
- 아직 강사 로그인 기능을 만들지 않아 파라미터로 instructorId를 받아 작동하도록 임시로 만들었습니다. - 추후에 강사 로그인 구현 후 Guard를 추가해 강의 등록 기능을 개선할 예정입니다.
- 현재는 Dto에서 유효성 검사를 간단하게 하도록 하였습니다. 추후 수정 예정
- 강사 정보 불러오기와 강의 포스트 기능을 간단하게 구현하였습니다.
- 설문(Survey) 응답 생성(Create) 및 추천(Recommend) API의 반환 타입을 SurveyDto에서 void 타입으로 수정 - 설문 응답 조회(Read) 시, 반환되는 데이터에 isMe 필드를 추가했습니다. 이를 통해 해당 응답을 현재 로그인한 사용자가 작성했는지 쉽게 식별 가능
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.
develop의 커밋메세지를 참고해주세요.