Skip to content

[FEAT/#116] 학생 대시보드 API 연동#117

Open
taegeon2 wants to merge 5 commits into
developfrom
feat/#116-student-dashboard-api-sync
Open

[FEAT/#116] 학생 대시보드 API 연동#117
taegeon2 wants to merge 5 commits into
developfrom
feat/#116-student-dashboard-api-sync

Conversation

@taegeon2
Copy link
Copy Markdown
Contributor

📝 요약

학생 대시보드의 이용한 제휴 내역 조회 및 건의함 작성 API를 연동합니다.

⚙️ 작업 내용

  • GET /students/partnerships/{year}/{month} 이용한 제휴 내역 연동
  • GET /suggestion/admin 건의 대상 관리자 목록 연동
  • POST /suggestion 건의 작성 연동
  • 개발 환경 목 핸들러 추가 (student.handlers.ts)

🔗 관련 이슈

Closes #116

✅ 체크리스트

  • 코딩 컨벤션(Biome/Lint)을 준수하였습니다.
  • 모든 타입 에러를 해결하였습니다. (Typecheck)
  • 변경 사항에 대한 테스트를 마쳤습니다.
  • 불필요한 로그(console.log)를 제거하였습니다. // 개발 모드에서만 켜놨습니다!

💬 리뷰어에게

  • 저희 작업 리스트 정리용으로 TASKS.md 만들었는데, 이거 활용하는거 어떨지 의견이 궁금합니다.
    특히 LLM에게 작업을 시킬 때 컨텍스트 관리하는데 유용하지 않을까 싶어서요..! 불필요하다 생각하시면 제거하겠습니다!
image

@taegeon2 taegeon2 self-assigned this May 25, 2026
@github-actions
Copy link
Copy Markdown

Thanks for the contribution!
I have applied any labels matching special text in your title and description.

Please review the labels and make any necessary changes.

@github-actions github-actions Bot added the feature 새로운 기능 구현 label May 25, 2026
@taegeon2 taegeon2 changed the title Feat/#116 student dashboard api sync [FEAT/#116] 학생 대시보드 API 연동 May 25, 2026
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

Comment thread src/widgets/map/model/useUserLocation.ts
Comment thread src/pages/student/suggestion/ui/StudentSuggestionPage.tsx Outdated
Comment thread src/shared/ui/kakao-map/KakaoMap.tsx
Comment thread src/widgets/map/ui/MapLocateButton.tsx
Comment thread src/entities/suggestion/lib/adapters.ts
Copy link
Copy Markdown
Contributor

@chunjaemin chunjaemin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Task.md를 어떤 배경에서 고려하게 된 것인지 궁금하네욥

@taegeon2
Copy link
Copy Markdown
Contributor Author

Task.md를 어떤 배경에서 고려하게 된 것인지 궁금하네욥

우선 UI작업한지가 좀 되어서 어떤 부분 연동을 해야하는지 기억이 안나서 확실히 암기해두려고 했고, 동시에 Claude와 같은 agent를 통해 작업할 때, 새로운 세션마다 맥락이 초기화될텐데(메모리에 남는다 해도 제한적), tasks.md를 LLM에게 첫 대화 시작에 넘긴 후 작업을 진행하면 더 토큰 소모도 줄이면서 작업 속도도 빨라질거 같다고 생각했습니다..!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

feature 새로운 기능 구현

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[FEAT/#116] 사용자 대시보드 API 연동

2 participants