-
Notifications
You must be signed in to change notification settings - Fork 6
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat-fe: 지원자 정렬 기능 #824
base: fe/develop
Are you sure you want to change the base?
feat-fe: 지원자 정렬 기능 #824
Conversation
1729131205.722389 |
1729131208.795249 |
1729131210.541489 |
1729131216.799889 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
렛서 고생 많으셨어요. useProcess
관련하여 정렬 조건을 부여했을 때 불필요한 query가 재발생하는 것을 막기 위해 refetch 로직을 따로 분리해주신 부분에 대해서도 감사히 설명 들었습니다.
현재는 오직 순 정렬 옵션만 존재해서, 역순 정렬을 제외하신 부분에 대해 의견을 여쭙고자 해요. 이밖에 코드 레벨에서 피드백 드릴 부분은 없는 상태여서 코멘트만 살짝 남겨봅니다 😄
const sortOptions: DropdownSortOptions = { | ||
sortByScore: '평점 높은 순', | ||
sortByCreatedAt: '지원 날짜 순', | ||
}; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
프로세스 관련 타입에는 정렬 옵션에 대해 ASC
와 DESC
가 모두 정의되어 있는데, 현재는 두 가지 모두 ASC
옵션만 존재하네요. 혹시 "평점 낮은 순"처럼 DESC
가 필요할 상황은 없을까요?
목적
작업 세부사항
참고사항
Dropdown sm 사이즈에 padding 값 변경
상하 패딩이
12px
로 설정되어 있는 부분을 피그마 시안에 맞춰8px
로 변경했습니다.프로세스 목록 요청 queryKey 설정
queryKey에 sortOption을 추가하면 예상치 못하게 기존 요청도 재요청이 가는 동작을 확인해 queryKey에서 제외하고 useEffect의 sortOption 을 의존성 배열로 추가하고 refetch되도록 처리해놨습니다.
일반적이지 않은 상황이라 주석을 남겼습니다.
SORT_APPL_01
closes #823