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

[Feat] 지원서 보기,수정,취소 기능 #1313

Merged
merged 59 commits into from
Mar 25, 2024

Conversation

PHJoon
Copy link
Contributor

@PHJoon PHJoon commented Mar 19, 2024

📌 개요

💻 작업사항

  • 공고페이지와 연결시켜놨습니다. 지원서 확인하기, 지원하기 버튼으로 테스트하실 수 있습니다.
  • ref로 관리하던 text input, checkbox의 값들을 recoil 상태 관리로 변경했습니다.
  • ref는 입력하지 않은 문항이 있는 경우 focus하기 위한 용도로만 사용합니다.
  • 지원서 페이지에서 modal, snackbar도 recoil 상태로 관리합니다.
  • 수정과 취소 버튼 위치가 애매해서 일단 헤더에 위치시켰습니다.(피드백 부탁드립니다)
  • 공고 메인으로 돌아가기와 뒤로가기 버튼도 위치할 곳이 마땅치 않아 Footer를 추가해서 위치시켰습니다.(피드백 부탁드립니다)

스크린샷 2024-03-20 오전 7 48 06 스크린샷 2024-03-20 오전 7 48 10 스크린샷 2024-03-20 오전 7 48 35 스크린샷 2024-03-20 오전 7 48 40 스크린샷 2024-03-20 오전 7 48 50 스크린샷 2024-03-20 오전 7 48 45

✅ 변경로직

  • 피드백 받은 내용대로 공고메인을 헤더로, 취소 수정버튼을 푸터로 옮겼습니다.
  • 기존 state로 관리하던 상태들을 다 없애고 form에서 submit할 때 formData를 가지고 처리하는 방식으로 훨씬 간결하고 쉽게 수정했습니다.(정연님 피드백 감사합니다)
  • 기존 applications 페이지 하나에서 => apply / my-application 페이지 두개로 나눴습니다.(apply = 지원 / my-application = 보기/취소/수정)
  • 기존 ref들이 form 개별 항목들 컴포넌트 내로 들어가서 inputRef들로 관리됐는데, 문항별로 div 태그에 ref를 걸어서 더 편하게 관리하는 식으로 바꿨습니다. focus시 해당 문항이 화면 스크롤 중앙에 위치하는 것도 추가했습니다.
  • text와 radio 문항에 남아있는 state는 유저가 제출한 폼을 수정하는 경우 기본값을 세팅하는데, default 값으로 설정하면 값을 바꿀 수가 없어서 이전과 같이 내부에서만 state로 관리하는 방식입니다.

스크린샷 2024-03-22 오전 2 19 51

PHJoon added 30 commits March 13, 2024 17:41
…분리 및 util 로직 분리 / applicationForm 폴더 분리
PHJoon added 21 commits March 21, 2024 20:21
@PHJoon
Copy link
Contributor Author

PHJoon commented Mar 21, 2024

피드백 감사합니다! 변경사항들은 본문의 변경로직에 적어놨습니다. 👍

Copy link
Member

@yoouyeon yoouyeon left a comment

Choose a reason for hiding this comment

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

확인 늦어서 죄송합니다!! 변경사항들 확인했어요.
줄바꿈 설정 하나만 다시 확인 부탁드리고 해결되면 바로 머지하셔도 될 것 같아요! 수고 많으셨습니다!!!! 👍

@@ -52,6 +52,7 @@
.questionText {
margin-bottom: 1rem;
font-size: 1rem;
word-break: keep-all;
Copy link
Member

Choose a reason for hiding this comment

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

여기인지 모르겠는데, 영어로 띄어쓰기 없이 긴 문자 적으면 줄바꿈이 잘 안되고 있습니다! break-all로 설정해야 되지 않을까 싶어요

Screen_Shot 2024-03-24 13 48 29

Copy link
Contributor Author

Choose a reason for hiding this comment

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

수정했습니다!

Copy link
Member

@hyobb109 hyobb109 left a comment

Choose a reason for hiding this comment

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

확인했습니다👍 고생 너무 많으셨습니다!!

@PHJoon PHJoon merged commit 2a9c00f into recruit Mar 25, 2024
@PHJoon PHJoon deleted the Feat/1279-지원서-보기수정취소 branch March 25, 2024 04:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
recruit 모집 기능 라벨 user 일반 사용자가 사용하는 서비스에서 발생한 issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

지원서 확인, 수정, 취소 기능
3 participants