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

v0.11.1 -> v0.11.2: take country into account when determining state fallback rate #83

Merged
merged 1 commit into from
Jan 21, 2025

Conversation

jcain3389
Copy link
Contributor

@jcain3389 jcain3389 commented Jan 16, 2025

Stakeholder Overview (learn more)

  • When testing the update of vertex_client to v0.11.1 in the checkout app, @lbeougher discovered a bug wherein an order shipping to Valladolid, Spain was being taxed at 5.65% (when it should've been taxed at 0%). Upon closer inspection, it was determined that this was the Virginia state tax, and it was being applied due to a collision in the state code ('VA', in this case). This PR fixes this pre-existing bug by taking a state's country into account when determine the proper fallback tax rate to apply.

Risk Estimate (learn more)

  • ✅ Negligible risk!

Changes

  • First check that the country code is domestic before determining which state tax rate to apply.

Monday Item

@jcain3389 jcain3389 force-pushed the fix-state-code-collision-bug branch from 2a82c08 to 9b1c9d1 Compare January 17, 2025 15:33
@jcain3389 jcain3389 marked this pull request as ready for review January 21, 2025 12:14
@jcain3389 jcain3389 requested a review from a team as a code owner January 21, 2025 12:14
@jcain3389 jcain3389 force-pushed the fix-state-code-collision-bug branch from 96723c1 to c598411 Compare January 21, 2025 15:23
@jcain3389 jcain3389 changed the title take country into account when determining state fallback rate v0.11.1 -> v0.11.2: take country into account when determining state fallback rate Jan 21, 2025
@jcain3389 jcain3389 merged commit e02f4fa into master Jan 21, 2025
7 checks passed
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

Successfully merging this pull request may close these issues.

2 participants