-
Notifications
You must be signed in to change notification settings - Fork 5
Feat: 지원현황, 공유하기 수정 #152
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: 지원현황, 공유하기 수정 #152
Conversation
…licationStatusCard-sorting
|
💄 Storybook: https://673dcf13c9a3a1189a7c6fd4-bihddyunbk.chromatic.com/ # Chromatic에서 생성된 Storybook URL |
cccwon2
left a comment
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 variants = { | ||
| orange: "bg-primary-orange-300 text-white hover:bg-primary-orange-200", | ||
| white: "bg-white text-primary-orange-300 border border-primary-orange-300 hover:bg-primary-grayscale-100", |
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.
bg-primary-grayscale-100 -> bg-grayscale-100 로 해주세용
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.
여기선 gray 필요없어서 제거할게용 !
hongggyelim
left a comment
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.
수고하셨습니다!
🚧 작업 내용
1. 지원현황
2. 공유하기 버튼
3. 로직 리펙토링
4. 공통 유틸 함수 utils에 정의
5. 지원자 소셜로그인 시 store값이 string으로 저장되어 정보수정유도 로직에 걸려서, 기본 x로 넣는 로직 추가하였습니다.
6. 파일명 수정 BookmarkBtn => ScrapBtn, 지원현황 ApplicationStatus => ApplyStatus
🐞 수정할 내용
- ✅ 스크랩