Skip to content
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

Fix/127 - 공고 등록/수정 이미지 관련 QC 사항 수정 #161

Merged
merged 6 commits into from
Jan 24, 2025

Conversation

MrMirror21
Copy link
Contributor

@MrMirror21 MrMirror21 commented Jan 24, 2025

Related issue 🛠

#127

Work Description ✏️

  • Step Indicator 변경으로 인한 공고 등록/수정 step indicator 디자인 깨짐 수정
  • 공고 등록 시 이미지 첨부만 되고 실제 서버에 등록되지 않던 현상 수정
  • 공고 수정 중 뒤로가기 시 등록한 이미지가 사라지던 현상 수정
  • 공고 수정 시 자동 입력 누락 항목 재적용(주소, 이미지)

(gif, 영상 업로드 모두 먹통이라 시연 영상은 유튜브 업로드 후 링크로 대체합니다.)
https://youtube.com/shorts/8HF2cR35jss

Uncompleted Tasks 😅

  • 기획, 서버, 디자인 측 공고 나이 제한 항목에 대한 결정 후 나이 제한 자동 입력 반영 예정

To Reviewers 📢

@MrMirror21 MrMirror21 added 🐞 BugFix Something isn't working ✨ Feature New feature or request 🎨 Html&CSS Markup & Styles labels Jan 24, 2025
@MrMirror21 MrMirror21 requested a review from naarang January 24, 2025 09:31
@MrMirror21 MrMirror21 self-assigned this Jan 24, 2025
Copy link
Contributor

@naarang naarang left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨습니다~!!

@@ -162,7 +153,7 @@ const EmployerLaborContractForm = ({

return (
<>
{isLoading && (
{isPending && (
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

오 훨씬 깔끔하네요!!👍

@MrMirror21 MrMirror21 merged commit 7c30c76 into main Jan 24, 2025
@MrMirror21 MrMirror21 deleted the fix/127-서류생성로딩처리 branch January 24, 2025 09:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐞 BugFix Something isn't working ✨ Feature New feature or request 🎨 Html&CSS Markup & Styles
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants