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

Add new values for TaxIDType and fix TaxIDTypeCHVAT #1017

Merged
merged 1 commit into from
Jan 23, 2020

Conversation

remi-stripe
Copy link
Contributor

r? @richardm-stripe
cc @api-libraries

@remi-stripe remi-stripe changed the title Add new values for TaxIDType and fix TaxIDTypeCHVAT Add new values for TaxIDType and fix TaxIDTypeCHVAT Jan 23, 2020
@remi-stripe remi-stripe merged commit a0ac5b8 into master Jan 23, 2020
@remi-stripe remi-stripe deleted the remi-add-new-tax-ids branch January 23, 2020 18:59
nadaismail-stripe added a commit that referenced this pull request Oct 18, 2024
* added config hash to setup

* resolving merge conflicts and adding test case

* updates based on pr comments

* fix for failed test case

* removed generic toast throwing and updates success error code check based on PR comments

* update for failing test classes

* fix for failing test

* Some nit fixes

* Nit changes

* Revert small change

* Fix syncing the config hash between saves

* potentially fix bug during setup phase

* missed merge detail

* fixed bug with data mapping losing values on back from sync preferences

* Trying to reduce some of the ambiguity in the code

* Remove a js error in the console

* fixed bad ref

* more bugs

* Pushing this change to see total diff and figure out what else to revert

* Reverting a few more changes for clarity

---------

Co-authored-by: arnoldezeolisa <[email protected]>
Co-authored-by: Nada Ismail <[email protected]>
Co-authored-by: Jacob Mather <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants