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: 지원경로 기타 클릭 후 값 입력 시 제출할 수 있도록 수정한다. #106

Merged
merged 4 commits into from
Mar 3, 2024

Conversation

bada308
Copy link
Collaborator

@bada308 bada308 commented Mar 2, 2024

주요 변경사항

  • channel이라는 배열에 channelchannelEtc의 값을 넣었기 때문에 지원경로 입력 확인 시 channel이라는 배열을 이용했습니다.

리뷰어에게...

관련 이슈

closes #103

@bada308 bada308 self-assigned this Mar 2, 2024
Copy link
Collaborator

@baegyeong baegyeong left a comment

Choose a reason for hiding this comment

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

👍

Copy link
Collaborator

@geongyu09 geongyu09 left a comment

Choose a reason for hiding this comment

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

좋아요!!

Copy link
Collaborator

@2yunseong 2yunseong left a comment

Choose a reason for hiding this comment

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

변경된 코드에서 다른 동작을 할 것 같습니다! 코멘트 꼭 체크해주세요.

Comment on lines +29 to 30
if (channel.length === 0) {
alert("지원 경로를 선택해주세요.");
Copy link
Collaborator

Choose a reason for hiding this comment

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

기존의 코드는 push가 되지않은 배열(local storage)의 상태에 접근하고,
지금의 코드는 분기문에서 push가 된 배열을 검사합니다. 의도하신 동작인가요?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

네!
기존 코드는 지원 경로에 기타를 선택하고 값을 입력하면, 제출할 때 "지원 경로를 선택해주세요"라는 alert가 뜹니다.
즉, channelEtc가 있어도 channel이 없다고 판별하고 있었습니다.

NextButton에서도 channelEtc가 존재하면 필수 항목을 모두 입력했다고 판단하고,
이 파일의 L45~48을 살펴보면 channel 이라는 name에 L27에 선언한 channel을 문자열 형태로 넘겨주는 것을 확인할 수 있습니다.

그래서 push가 된 배열로 channel의 입력 여부를 판단하는 게 맞다고 생각해 위와 같이 로직을 변경했습니다!

@bada308 bada308 requested a review from 2yunseong March 2, 2024 15:28
Copy link
Collaborator

@2yunseong 2yunseong left a comment

Choose a reason for hiding this comment

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

좋습니다!
혹시나 해서 확인 한 번 드렸어요 !! 고생하셨습니다~!!

@2yunseong 2yunseong merged commit 10b1c2e into main Mar 3, 2024
@2yunseong 2yunseong deleted the fix/103-channel-etc branch September 9, 2024 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[FE] bug: 지원서 작성 시 지원경로 기타만 선택 시 오류 발생
4 participants