-
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: 지원자 이름 검색 #814
feat-fe: 지원자 이름 검색 #814
Conversation
1728980488.876559 |
1728980498.574779 |
1728980672.188229 |
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.
렛서 고생하셨어요. 로컬에서 검색 기능이 의도대로 잘 동작하는 것을 확인했어요. useSearchApplicant
훅을 components/dashboard
경로로 위치시킨 결정에 대해서도 동의합니다. Approve 드릴게요.
() => | ||
process.applicants.filter(({ applicantName, isRejected }) => { | ||
const matchesName = searchedName ? applicantName.includes(searchedName) : true; | ||
const matchesRejection = showRejectedApplicant ? isRejected : !isRejected; |
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.
지금도 잘 동작하는 로직이지만, 아래와 같이 간략화가 가능할 것 같아요.
const matchesRejection = showRejectedApplicant ? isRejected : !isRejected; | |
const matchesRejection = showRejectedApplicant === isRejected; |
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.
반영했습니다! 꼼꼼하게 봐주셔서 감사해요.
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.
굳 쟙 lesser
const handleName = useCallback((newName: string) => { | ||
setName(newName); | ||
}, []); |
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.
함수 네이밍에서 함수의 역할을 잘 유추가 안되었어요! 그럼에도 함수 자체가 복잡한 기능을 가지진 않는 것 같아 리네이밍 하는 건 렛서의 몫으로 남겨두도록 하겠습니다~!
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.
updateName으로 변경했습니다~
목적
구현 사항
참고사항
ProcessColumn을 메모이제이션 했었지만 element가 적은 상황에서는 오히려 committed at(DOM 요소가 표시되기까지의 시간)이 늘어났습니다. 따라서 따로 렌더링 최적화는 진행하지 않았습니다!
SRCH_01
closes #813