Conversation
| includeWithdrawn: request.includeWithdrawn, | ||
| onlyEnrolled: request.onlyEnrolled, |
There was a problem hiding this comment.
sdk와는 관련이 없지만 이것을 보고 알게 된건데 생각해보니 서버에서 요청을 할때 github 관련 정보나 전공 등 최근 추가된 파라미터들론 검색을 할수가 없던데 이런것들을 서버에서 작업해야 할까요?
There was a problem hiding this comment.
예를 들어 어떤 방식을 말씀하시는 건가요??
There was a problem hiding this comment.
특정 전공인 학생을 검색한다거나, 그런 것들을 말씀하신 건가요?
There was a problem hiding this comment.
검색 파라미터로 전공 문자열,github ID를 전달할 수 있도록 하는 것을 말한겁니다!
예를 들어 junjun을 파라미터로 전달하면 3415 전준연이 조회되도록 하는 그런 식으로요
There was a problem hiding this comment.
특정 전공(예: 프론트엔드)을 검색했을 때 해당 전공의 학생 목록이 반환되는 기능은 충분히 활용 가치가 있을 것 같습니다. 그런데 개인적으로는 GitHub ID로 학생을 검색하는 경우는 그렇게 많지 않을 것 같습니다. 그래도 기능이 있다면 특정 상황에서는 유용하게 활용될 수 있을 것 같네요!
There was a problem hiding this comment.
작업을 진행한다면 오늘 내로 작업이 가능할 것 같긴 한데 우선 진행해보겠습니다.
There was a problem hiding this comment.
특정 전공(예: 프론트엔드)을 검색했을 때 해당 전공의 학생 목록이 반환되는 기능은 충분히 활용 가치가 있을 것 같습니다. 그런데 개인적으로는 GitHub ID로 학생을 검색하는 경우는 그렇게 많지 않을 것 같습니다. 그래도 기능이 있다면 특정 상황에서는 유용하게 활용될 수 있을 것 같네요!
서버 측 작업 진행하였습니다.
코드베이스 확인 결과 전공이 누락된게 아니라 학과가 누락되어 있었어서 이를 수정하였습니다
개요 💡
최신 서버 변경사항을 SDK에 반영해 요청 파라미터와 응답 타입을 서버 스펙과 일치시키는 것이 목적입니다.
기존 타입 정의를 공통 응답 포맷으로 정리해 유지보수성과 일관성을 높였습니다.
작업내용 ⌨️
clubStatus,foundedYear추가includeWithdrawn,onlyEnrolled추가ClubStatus추가,ClubSortBy확장Club,Student모델 필드 최신 스펙에 맞게 optional/신규 필드 반영GetClubsResponse,GetStudentsResponse,GetProjectsResponse,GetMealsResponse,GetSchedulesResponse,HealthCheckResponse를ApiResponse기반으로 정리ClubStatus등) export 반영리뷰 요청사항 👀