[FEAT/#116] 학생 대시보드 API 연동#117
Conversation
|
Thanks for the contribution! Please review the labels and make any necessary changes. |
There was a problem hiding this comment.
Code Review
This pull request integrates backend APIs for student suggestions and partnership benefits, and implements a WebView-based KakaoMap component with location tracking capabilities. The review comments highlight several important issues: a potential memory leak and battery drain in useUserLocation due to unhandled asynchronous cleanup on unmount; a hardcoded current year in StudentSuggestionPage that breaks queries when navigating across years; incorrect rendering of the user's location overlay at the default map center before actual coordinates are fetched; a lack of visual feedback for the disabled state of the map locate button; and a potential ID mismatch when submitting suggestions for different administrative targets. All review comments are valid and have been retained.
chunjaemin
left a comment
There was a problem hiding this comment.
Task.md를 어떤 배경에서 고려하게 된 것인지 궁금하네욥
우선 UI작업한지가 좀 되어서 어떤 부분 연동을 해야하는지 기억이 안나서 확실히 암기해두려고 했고, 동시에 Claude와 같은 agent를 통해 작업할 때, 새로운 세션마다 맥락이 초기화될텐데(메모리에 남는다 해도 제한적), tasks.md를 LLM에게 첫 대화 시작에 넘긴 후 작업을 진행하면 더 토큰 소모도 줄이면서 작업 속도도 빨라질거 같다고 생각했습니다..! |
📝 요약
학생 대시보드의 이용한 제휴 내역 조회 및 건의함 작성 API를 연동합니다.
⚙️ 작업 내용
🔗 관련 이슈
Closes #116
✅ 체크리스트
💬 리뷰어에게
특히 LLM에게 작업을 시킬 때 컨텍스트 관리하는데 유용하지 않을까 싶어서요..! 불필요하다 생각하시면 제거하겠습니다!