강의 정보 반환 시 snuttev 정보 함께 전달

wafflestudio / snutt-timetable

SNUTT timetable service

14 stars 0 forks source link

Closed Hank-Choi closed 3 months ago

Hank-Choi commented 3 months ago

이름 path 등 마음에 안드는 것들이 많아서 추천 부탁

테스트 안해봤고 https://github.com/wafflestudio/snutt-ev/pull/103 이거 머지 이후 테스트, 수정 커밋 예정이므로 위 PR 부터 선 리뷰 부탁드립니다.

davin111 commented 3 months ago

테스트 깨져있으면 작업 중이구나~ 생각하게 되어서, pass 상태로 올려주면 더 리뷰가 빠를 듯 합니다!

Hank-Choi commented 3 months ago

이건 ev dev 서버 배포하고 할거라 보류중임

Hank-Choi commented 3 months ago

하는 김에 기존에 snutt조회 후 강의번호, 교수이름으로 조회하던 ev id를 lecture, timetableLecture 기본 응답으로 포함

search에서만 평점평균까지 내려주고 나머지는 ev id만 내려줌

timetableLecture에서 사실 변경할 필요 없고 성능이 약간 낮아지지만 변경한 이유는 단지 search api랑 통일성을 높이기 위함

Hank-Choi commented 3 months ago

@wafflestudio/snutt-server 리뷰부탁