search

sparcs-kaist / otlplus-server

OTL Plus의 NestJS 마이그레이션입니다.
https://otl.kaist.ac.kr
5 stars 0 forks source link

feat: /api/users/:id/wishlist/add-lecture API #68

Closed doxylee closed 8 months ago

doxylee commented 8 months ago

특이사항

  1. src/common/schemaTypes/types.ts 에 LectureMedium 추가. toJsonLecture가 LectureMedium 혹은 LectureDetails를 받도록 만들어져 있지만, 전자의 경우 타입이 만들어지지 않아 추가했습니다. 이름 추천 받습니다. LectureSimple로 할까 했지만 Basic보다도 간단(?)하게 느껴져 일단은 Medium이라고 이름 붙여두었습니다.
  2. 기존 toJsonLecture에서 NESTED에 따라 입력받는 타입이 달라지도록 되어있는데, 제대로 작동하지 않는 것 같아 고쳐두었습니다. 이를 위해 src/common/schemaTypes/types.ts 에 타입 가드 함수를 추가했습니다.
doxylee commented 8 months ago

@ddungiii @duncan020313 변경 내용 getLectures API 작동하도록 수정했는데 확인 부탁드립니다. group filter 부분이 로직이 잘못되었던 것 같습니다

django 버전