-
Notifications
You must be signed in to change notification settings - Fork 2
feat: 지원하기 플로우 리뉴얼 #314
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
Merged
Merged
feat: 지원하기 플로우 리뉴얼 #314
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
- 지원 플로우에 funnel 구조 도입을 위함
- 불필요한 옵셔널 props 제거 - 타입 가드를 제거하고 런타임에서 직접 검사할 수 있는 방식
- current 유무에 따라 Step의 렌더링을 결정 - jobFamily가 있다면 데이터 객체를 사용 - 지원 플로우, 핀 재설정, 이어서 작성하기 플로우 모두 공통적으로 사용하는 컴포넌트
- 이메일 입력 폼 - 인증번호 입력 폼
- STEP을 동적으로 할당할 때 사용
- 타이머와 같은 부수적인 로직은 커스텀 훅으로 전환 - 불필요하게 선언된 외부 의존성 컴포넌트 제거
- 순수 함수를 통해 명시적으로 상태를 드러내기 위함 - JSX 내에서 getter 함수의 가독성 저하 문제 해결
- 개인정보 수집 및 이용 동의 - 회비 및 보증금 납입 정책 동의
- CheckBoxGroup 추가 - validation 조건(정책)을 명시적으로 선언 - mutation 호출 시 불필요한 isPending 값 제거(다음 Step으로 이동하기 때문)
- InputArea를 사용하는 TextQuestionField - TextField를 사용하는 UrlQuestionField
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
Member
|
고생하셨습니다! 지금 일정이 빠듯한 것 같아서 아직 자세히 확인은 못 했는데 먼저 approve 후에 확인해 보겠습니다! |
- 브라우저 마다 다르게 적용되는 속성 제거
- relative일 때도 같은 위치에 적용하기 위함
- ContinueWritingFunnel에서 URL 파라미터 대신 /apply/status API로 실제 상태 확인 - 이미 제출 완료/이어서 작성할 내용이 없는 경우 안내 후 리다이렉트 - tempSavedStep URL 파라미터 및 관련 prop 제거
kimdonggu42
approved these changes
Dec 28, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
💡 작업 내용
💡 자세한 설명
Funnel 구조 도입
Apply 도메인 API 리펙토링
Apply 공통
공통
📗 참고 자료 (선택)
📢 리뷰 요구 사항 (선택)
✅ 셀프 체크리스트
closes #293