Skip to content

[feat] 서버 최신 스펙 반영 (동아리/학생 필터, 응답 타입 정리)#2

Open
junjuny0227 wants to merge 2 commits intomainfrom
feat/club-student-filters-and-types-update
Open

[feat] 서버 최신 스펙 반영 (동아리/학생 필터, 응답 타입 정리)#2
junjuny0227 wants to merge 2 commits intomainfrom
feat/club-student-filters-and-types-update

Conversation

@junjuny0227
Copy link
Copy Markdown
Member

개요 💡

최신 서버 변경사항을 SDK에 반영해 요청 파라미터와 응답 타입을 서버 스펙과 일치시키는 것이 목적입니다.
기존 타입 정의를 공통 응답 포맷으로 정리해 유지보수성과 일관성을 높였습니다.

작업내용 ⌨️

  • 동아리 조회 파라미터 확장
    • clubStatus, foundedYear 추가
  • 학생 조회 파라미터 확장
    • includeWithdrawn, onlyEnrolled 추가
  • 타입 정의 최신화
    • ClubStatus 추가, ClubSortBy 확장
    • Club, Student 모델 필드 최신 스펙에 맞게 optional/신규 필드 반영
  • 응답 타입 공통화
    • GetClubsResponse, GetStudentsResponse, GetProjectsResponse, GetMealsResponse, GetSchedulesResponse, HealthCheckResponseApiResponse 기반으로 정리
  • 인덱스 export 정리
    • 외부 사용 가능 타입(ClubStatus 등) export 반영

리뷰 요청사항 👀

  • 이번 변경이 최신 서버 스펙과 정확히 일치하는지 확인 부탁드립니다.

Comment on lines +22 to +23
includeWithdrawn: request.includeWithdrawn,
onlyEnrolled: request.onlyEnrolled,
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

sdk와는 관련이 없지만 이것을 보고 알게 된건데 생각해보니 서버에서 요청을 할때 github 관련 정보나 전공 등 최근 추가된 파라미터들론 검색을 할수가 없던데 이런것들을 서버에서 작업해야 할까요?

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

예를 들어 어떤 방식을 말씀하시는 건가요??

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

특정 전공인 학생을 검색한다거나, 그런 것들을 말씀하신 건가요?

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

검색 파라미터로 전공 문자열,github ID를 전달할 수 있도록 하는 것을 말한겁니다!
예를 들어 junjun을 파라미터로 전달하면 3415 전준연이 조회되도록 하는 그런 식으로요

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

특정 전공(예: 프론트엔드)을 검색했을 때 해당 전공의 학생 목록이 반환되는 기능은 충분히 활용 가치가 있을 것 같습니다. 그런데 개인적으로는 GitHub ID로 학생을 검색하는 경우는 그렇게 많지 않을 것 같습니다. 그래도 기능이 있다면 특정 상황에서는 유용하게 활용될 수 있을 것 같네요!

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

작업을 진행한다면 오늘 내로 작업이 가능할 것 같긴 한데 우선 진행해보겠습니다.

Copy link
Copy Markdown
Member

@snowykte0426 snowykte0426 Mar 29, 2026

Choose a reason for hiding this comment

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

특정 전공(예: 프론트엔드)을 검색했을 때 해당 전공의 학생 목록이 반환되는 기능은 충분히 활용 가치가 있을 것 같습니다. 그런데 개인적으로는 GitHub ID로 학생을 검색하는 경우는 그렇게 많지 않을 것 같습니다. 그래도 기능이 있다면 특정 상황에서는 유용하게 활용될 수 있을 것 같네요!

서버 측 작업 진행하였습니다.
코드베이스 확인 결과 전공이 누락된게 아니라 학과가 누락되어 있었어서 이를 수정하였습니다

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants