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-fe: 지원자 지원시 confirm 여러번 나타나는 버그 수정 #996

Merged
merged 6 commits into from
Jan 22, 2025

Conversation

github-actions[bot]
Copy link
Contributor

@github-actions github-actions bot commented Jan 16, 2025

Original issue description

목적

지원자 지원시 confirm 여러번 나타나는 버그

작업 세부사항

  • 지원자 지원시 confirm 두번 나타나는 버그
    -> 해당 버그는 개발환경에서는 안나타나는데, 배포 이후 나타나는 현상이에요.

아래는 로컬환경
화면-기록-2025-01-16-오후-2 06 37
아래는 번들링 이후 beta환경
화면-기록-2025-01-16-오후-2 06 37

참고 사항

아래의 별표줄 밑에 요구사항 ID만 작성해주세요. Prefix 금지!


fix-unexpected-confirm

closes #995

기능 구현 목록

  • 기본정보에 해당하는 값, LocalStorage 저장 타이밍 수정
    • error값이 있는 경우 저장하지 않도록 만들었습니다. 에러가 저장되면 validate 로직을 피해 value 값으로 저장되는 부분이 있었기 때문입니다.
    • useEffect로 구현하였습니다.
    • 기본 정보에 해당하는 Input 컴포넌트를 아래로 분리하였습니다. src/component/recruitmentPost/ApplyForm/index.tsx
  • Confirm을 두 번 물어보지 않게 만들었습니다.

추가 공유사항

  1. 현재 StrictMode에서는 올바르게 작동하지 않습니다. confirm이 무조껀 true를 반환합니다. -> 배포상황에선 정상 작동합니다.
  2. 이걸 어떻게 해결해야할지 모르겠습니다.
    • 모듈을 불러오는 타이밍에 한 번 실행하기 -> 컴포넌트 내부에 변수값이 있음
    • useEffect로 실행하기 -> 안됨
    • 👉 그냥 StrictMode에선 confirm을 두 번 실행하는 걸로 구현해놨습니당

@github-actions github-actions bot added fix 버그 수정 frontend 프론트엔드 labels Jan 16, 2025
@lurgi lurgi requested a review from llqqssttyy January 17, 2025 06:50
@lurgi lurgi marked this pull request as ready for review January 17, 2025 06:51
Copy link
Contributor Author

1737096671.099769

Copy link
Contributor Author

1737096675.263689

Copy link
Contributor Author

1737096679.621469

@lurgi lurgi merged commit bcd23ba into fe/develop Jan 22, 2025
10 of 11 checks passed
@lurgi lurgi deleted the 995-fe-x-unexpected-confirm branch January 22, 2025 05:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
fix 버그 수정 frontend 프론트엔드
Projects
Status: 완료
Development

Successfully merging this pull request may close these issues.

2 participants