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

[Bug] Mobile phone validation is incorrectly displaying as invalid when the phone number is valid #464

Open
pbojanczyk opened this issue Mar 14, 2024 · 1 comment

Comments

@pbojanczyk
Copy link

Library used

paypal-js

🐞 Describe the Bug

The form validation incorrectly triggers validation errors when the form is in a valid state. The only was to get around it is to erease one character and re-type it.

🔬 Minimal Reproduction

Reproduction is very easy. You can even observe the issue on the Official PayPal demo site: https://developer.paypal.com/demo/checkout/#/pattern/validation

  1. I was using US as my country but I was also able to replicate this by setting Canada as my country
  2. Enter 416 555 5555 into the Mobile text box
  3. Everything should be fine so far but try adding one more digit into the phone.
  4. Paypal textbox entry rejects the digit and it will still display 416 555 5555 however the textbox is now in errornous state

😕 Actual Behavior

Paypal textbox entry rejects the digit and it will still display 416 555 5555 however the textbox is now in errornous state

🤔 Expected Behavior

Paypal textbox entry should reject the digit and the phone number field should remain in a valid state

🌍 Environment

  • Node.js/npm: -
  • OS: - Windows
  • Browser: - Chrome

➕ Additional Context

Add any other context about the problem here.

@wsbrunson
Copy link
Member

hi @pbojanczyk, thank you for your report.

The team that works on this component is aware of the issue. They will link their internal jira ticket to this issue and update it when they deploy a fix.

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

No branches or pull requests

3 participants
@wsbrunson @pbojanczyk and others