search

sparcs-kaist / otlplus-server

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

feat: department-options, favorite-departments API #73

Closed doxylee closed 8 months ago

doxylee commented 8 months ago

리뷰 필요 사항

  1. /session 아래의 API이긴 한데 실제 동작은 각각 다른 모듈에 속해있는 것 같아 아래처럼 호출하고 dependency가 생기게 만들었습니다. 이렇게 해도 될지, 더 좋은 방법이 있을지 의견 주시면 감사하겠습니다.
    • /session/department-options -> SessionController -> DepartmentsService -> DepartmentsRepository
    • /session/favorite-departments -> SessionController -> SessionService -> UserRepository
  2. 기존에 /session/department-options API가 department별로 exist 쿼리를 한 번씩 날려서 3초 이상 걸리도록 비효율적으로 짜여져 있었는데, 이를 raw query를 이용해 효율화 했습니다. 다만 테이블 변경이 있을 시에 이 부분을 유의해야겠습니다.