Repository navigation
[admin] 지원자 목록 컬럼 헤더 클릭 정렬 추가 - #562
Merged
Merged
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
s2yeons
approved these changes
Oct 20, 2026
There was a problem hiding this comment.
- Overview: 어드민 지원자 목록에서 컬럼 헤더 클릭 시 서버 정렬(
sortBy,sortDirection)을 지원하는 PR입니다. 헤더 UI, URL 상태 동기화, API 파라미터 전달, 정렬 토글 로직과 테스트를 추가했습니다. - Intent: 지원자 목록의 정렬 UX를 개선하고, 정렬 상태를 URL에 반영해 공유·뒤로가기·SSR에서도 동일한 결과를 유지하려는 목적입니다.
- Risk: 중간. 서버 정렬 계약, URL 파싱/SSR 일관성, 쿼리 캐시 키 변경, 페이지 리셋/필터 유지 정책, 헤더 접근성 및 툴립 동작을 중점적으로 확인해야 합니다. 실제 정렬 정확성은 stage 환경 검증이 필요합니다.
0 inline comment(s)
This branch was successfully deployed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
개요 💡
어드민 지원자 목록에서 컬럼 헤더를 클릭하면 해당 컬럼 기준으로 전체 지원자를 정렬하도록 하였습니다. 목록은 10건씩 페이지 단위로 받아오므로 정렬은 서버(themoment-team/hellogsm-server-26#454)에서 하고, 프론트는 헤더 UI와 정렬 파라미터 전달을 맡습니다. stage 서버 OpenAPI에
sortBy,sortDirection이 반영된 것을 확인하였습니다.작업내용 ⌨️
API·타입
packages/types에OneseoSortBy,SortDirection타입을 추가하였습니다.getSearchedOneseoListURL·쿼리 키와useGetOneseoList에sortBy,sortDirection을 추가하였습니다. 값이 없으면 파라미터를 생략합니다.URL 동기화
sort(OneseoSortBy),order(asc|desc)로 저장합니다. 예:/?sort=NAME&order=descsort는 정렬 없음,order가 없거나 잘못되면 오름차순으로 봅니다. SSR 초기 조회도 같은 정렬로 받아옵니다.헤더
getNextSort로 분리하였습니다. 정렬되지 않은 컬럼은 오름차순 → 내림차순 → 해제 순으로 바뀌고, 다른 컬럼을 누르면 이전 정렬은 해제되고 새 컬럼을 오름차순으로 정렬합니다. 단위 테스트 3개를 추가하였습니다.ApplicantTH를 컬럼 목록 기반으로 바꾸고 헤더 텍스트를<button>으로 감싸 키보드로 조작할 수 있게 하였습니다.th에는aria-sort를 지정하고, 아이콘은 기본↕(흐리게) /▲/▼로 표시합니다.서류 제출 여부·입학 동의서 제출 여부의?툴팁은 정렬 버튼 밖에 두어, 툴팁을 보거나 눌러도 정렬되지 않습니다. 툴팁마다 있던 hover 상태는HeaderTooltip으로 옮겼습니다.필터 초기화와 탭 이동은 정렬을 유지합니다.관련 이슈 🚨
리뷰 요청사항 👀
aria-sort를 확인하였습니다.필터 초기화에서 정렬도 해제해야 할지 의견 부탁드립니다. 정렬은 필터가 아니라고 보고 유지하였습니다.