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

Upgrading to latest ember #34

Closed
pdud opened this issue Jan 17, 2023 · 7 comments
Closed

Upgrading to latest ember #34

pdud opened this issue Jan 17, 2023 · 7 comments

Comments

@pdud
Copy link
Contributor

pdud commented Jan 17, 2023

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.

  1. Adding github actions CI
  2. Upgrading to the ember 3.28, which is a breaking change because it drops node support.
  3. Upgrading to octane/ glimmer/ removing deprecations for 4.0

Is this something you would be interested in?

@arenoir
Copy link
Owner

arenoir commented Jan 17, 2023

Yes absolutely. Thank you

@pdud
Copy link
Contributor Author

pdud commented Jan 17, 2023

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!

@arenoir
Copy link
Owner

arenoir commented Jan 17, 2023

Thanks @pdud, let me know and I will publish a new version to npm.

@veelci
Copy link
Contributor

veelci commented Oct 17, 2023

@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.

@pdud
Copy link
Contributor Author

pdud commented Oct 24, 2023

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?

@veelci
Copy link
Contributor

veelci commented Oct 24, 2023

@pdud. Sure - I've opened #40 that fixes this and also strips spaces out from values being passed back via onUpdate()

@arenoir arenoir closed this as completed Oct 25, 2023
@pdud
Copy link
Contributor Author

pdud commented Oct 25, 2023

Thank you both for your work here! 👏

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