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

[FE] 계좌번호 입력 확인 버튼 활성화가 되지 않는 버그, 빈 값일 때 새로운 값이 반영되지 않는 버그 #602

Merged
merged 2 commits into from
Sep 26, 2024

Conversation

jinhokim98
Copy link
Contributor

@jinhokim98 jinhokim98 commented Sep 24, 2024

issue

구현 사항

조건문을 변경했습니다.
서버 상태가 아니라 클라이언트 상태가 빈 값인지를 확인해서 현재 입력 중인 값이 빈 값인지 검증해요

 const existEmptyField = bankNameState.trim() === '' || accountNumberState.trim() === '';

여기도 마찬가지.. 상태 헷갈림입니다
새로운 상태가 빈 값인지를 체크해서 초기에 빈 값일 때 새로운 값이 반영이 되지 않는 버그를 수정했어요

 if (bankNameState.trim() !== '' && bankName !== bankNameState) {
      changedField.bankName = bankNameState;
    }

    if (accountNumberState.trim() !== '' && accountNumber !== accountNumberState) {
      changedField.accountNumber = accountNumberState;
    }

@jinhokim98 jinhokim98 added this to the lev4 milestone Sep 24, 2024
@jinhokim98 jinhokim98 self-assigned this Sep 24, 2024
Copy link

@Todari
Copy link
Contributor

Todari commented Sep 24, 2024

PR 컨벤션좀요~!

@jinhokim98 jinhokim98 changed the title fix: can submit 조건이 잘못됐던 버그 수정 [FE] 계좌번호 입력 확인 버튼 활성화가 되지 않는 버그 Sep 24, 2024
@jinhokim98
Copy link
Contributor Author

급하게 올리느라 깜빡했습니다...

Copy link

@jinhokim98 jinhokim98 changed the title [FE] 계좌번호 입력 확인 버튼 활성화가 되지 않는 버그 [FE] 계좌번호 입력 확인 버튼 활성화가 되지 않는 버그, 빈 값일 때 새로운 값이 반영되지 않음 Sep 24, 2024
@jinhokim98 jinhokim98 changed the title [FE] 계좌번호 입력 확인 버튼 활성화가 되지 않는 버그, 빈 값일 때 새로운 값이 반영되지 않음 [FE] 계좌번호 입력 확인 버튼 활성화가 되지 않는 버그, 빈 값일 때 새로운 값이 반영되지 않는 버그 Sep 24, 2024
Copy link
Contributor

@soi-ha soi-ha 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
Contributor

@pakxe pakxe left a comment

Choose a reason for hiding this comment

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

버그 고치느라 고생많았어요~!! 덕분에 안정적인 서비스가 될 것 같네요 😆
이건 그냥 제안이긴한데 버그 내용을 더 직관적으로 이해할 수 있도록 버그가 발생하는 영상을 올려주면 좋을 것 같기도 해요~!!

p.s..파일체인지스가 적어서 읽기 좋네요.. 👍

@Todari Todari modified the milestones: lev4, v2.0.0 Sep 25, 2024
Copy link
Contributor

@Todari Todari left a comment

Choose a reason for hiding this comment

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

react-query에서 불러온 값이랑 비교해서 생긴 오류였군용
고생했어요~

@soi-ha soi-ha merged commit ef24cea into fe-dev Sep 26, 2024
2 checks passed
@soi-ha soi-ha deleted the feature/#599 branch September 26, 2024 06:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: ✅ Done
4 participants