search

sparcs-kaist / otlplus-server

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

refactor: get courseuser #78

Closed ddungiii closed 7 months ago

ddungiii commented 7 months ago

  1. 기존 courseDetails.include에 courseuser: true가 포함되어, 쿼리에서 모든 course에 대한 courseuser를 불러와 성능 저하가 있었습니다. courseDetails.include에서 courseuse: true를 제거하고, isUserSpecificRead를 만들어 사용합니다. -> 엄밀하게 다 측정해보지는 않았으나, 로컬에서 최대 5s -> 1s 로 성능 개선 확인

  2. courses.controller, courses.service에서 repository instance를 소문자로 시작하도록 변경, 필요한 함수명을 수정합니다.

  3. 필요한 API에 누락되었던 @Public을 추가합니다.

ddungiii commented 7 months ago

https://github.com/sparcs-kaist/otlplus-server/pull/77#discussion_r1505788927 의 후속작업입니다. @doxylee

ddungiii commented 7 months ago

@LarryKwon 리뷰 부탁드립니당