-
Notifications
You must be signed in to change notification settings - Fork 14
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
Upgrading to latest ember #34
Comments
Yes absolutely. Thank you |
Thanks so much for your interest! I'll continue with my ember 4.0 upgrade and make sure it's got all it needs to be ready for 4.0. I'll loop back at a later date. Thanks again! |
Thanks @pdud, let me know and I will publish a new version to npm. |
@pdud I'm currently running my app with your changes and I've noticed that the "onUpdate" argument of the InputCreditCardNumber component is called for invalid credit card numbers, but the value of the input component is not updated. You should be able to reproduce this by inputting an extra 1 after "4111111111111111" and should be able to see that "41111111111111111" (with the extra 1) is passed back via onUpdate. |
Thanks for reporting @veelci! I expect to have capacity to get this fixed in November. Do you want to try put in a PR before then? |
Thank you both for your work here! 👏 |
Hey @arenoir!
We love your add-on and have been using it for years! Thanks so much! 🙌
We're in the midsts of upgrading to Ember 4 and found some add ons that required updating including this one.
I've got three PRs ready to go for ember-credit-cards that can be seen passing on my fork.
Is this something you would be interested in?
The text was updated successfully, but these errors were encountered: