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

#216 [[Order.VATRegistrationNumber]] independent of user.TaxExempt #218

Merged
merged 1 commit into from
Mar 24, 2020
Merged

#216 [[Order.VATRegistrationNumber]] independent of user.TaxExempt #218

merged 1 commit into from
Mar 24, 2020

Conversation

Navilois
Copy link
Contributor

No description provided.

@WillStrohl
Copy link
Member

What's the purpose of this PR? There isn't an Issue associated and I don't see what problem this update is supposed to fix. The TaxExempt status is something that generally should be checked prior to evaluating and using the string value.

@Navilois
Copy link
Contributor Author

Of course there's an issue associated. See #216

When using guest checkout and the EU tax exemption system using VATNumbers, TaxExempt will never be true.

@WillStrohl
Copy link
Member

Ah... I missed that. Your explanation and the issue make sense. Thank you. :)

@WillStrohl WillStrohl added this to the 03.03.00 milestone Mar 24, 2020
@WillStrohl WillStrohl merged commit 1bb16ce into HotcakesCommerce:development Mar 24, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants