-
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: 지원자 카드에 평가 등록 여부, 평점 및 평가 등록수 추가 #493
Conversation
1723953640.628559 |
1723953648.381079 |
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.
코드 멋지게 구현해주셨네용~ 감사합니다~
/** | ||
* 현재는 API 스펙상 지원자 카드에 평점(averageScore)을 보여줄 방법이 아직 없습니다. | ||
* 따라서 평가자 수를 기준으로 '평가 대기 중', '평가 완료'의 2가지 플래그로 우선 적용했습니다. | ||
* 추후 API 스펙 변경시 이에 대한 PR을 별도로 올릴 예정입니다. | ||
* - 2024-08-18 by 아르 | ||
*/ | ||
// const evaluationString = averageScore ? `★ ${averageScore.toFixed(1)}` : '평가 대기 중'; |
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.
스토리북으로 확인해 보니 컴포넌트 너무 깔끔하고 좋네요~
대시보드가 한층 더 생기있어져 보는 재미가 있는 서비스가 된 것 같습니다.
작업 고생하셨어요!
Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com> Co-authored-by: Seongjin Hong <[email protected]> Co-authored-by: Seongjin Hong <[email protected]>
Original issue description
목적
작업 세부사항
참고 사항
현재는 API 설계상 심사 단계별 평점을 지원자 카드에서 바로 보여줄 방법이 없는 상태입니다. (API 수정 필요)
따라서 일단은 "평가 대기 중" / "평가 완료" 플래그만 추가할 예정입니다. API 스펙이 변경되면 이를 반영하기 위한 PR을 별도로 생성하겠습니다.
구현 전후 비교 가능한 이미지를 첨부합니다. 좌측이 구현 전, 우측이 구현 이후입니다.
COM_REC_CARD_02
목적
변경 사항
averageScore
)이 포함된 process mock data 필드를 새로 만들었습니다.<img>
요소에서 누락된alt
정보를 추가했습니다.변경 전 화면
변경 후 화면
참고 사항
averageScore
)을 지원자 카드에서 바로 보여주기가 어렵습니다. (API 수정 필요)지원자별 평점 조회 기능 추가 후 화면
관련 이슈