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

Fix USPS Priority Mail to Canada #8041

Merged
merged 1 commit into from
Mar 9, 2017
Merged

Conversation

jaywilliams
Copy link
Contributor

@jaywilliams jaywilliams commented Jan 4, 2017

As USPS states in the XML response:

The Origin ZIP Code and the Destination Postal Code is required for Priority Mail International when mailing to Canada.

This patch adds the three required fields, therefore fixing the issue with shipping USPS Priority Mail International from the US to Canada.

See Also: #3186

As USPS states in the XML response:
> The Origin ZIP Code and the Destination Postal Code is required for Priority Mail International when mailing to Canada.

This patch adds the three required fields, therefore fixing the issue with shipping USPS Priority Mail International from the US to Canada.
@vrann vrann self-assigned this Mar 6, 2017
@vrann vrann added this to the March 2017 milestone Mar 6, 2017
@magento-team magento-team merged commit 69a0f5c into magento:develop Mar 9, 2017
@vrann
Copy link
Contributor

vrann commented Mar 9, 2017

@jaywilliams Thank you for the contribution!

@jaywilliams jaywilliams deleted the patch-2 branch March 9, 2017 15:09
@jaywilliams
Copy link
Contributor Author

A merged pull request?!?! Hurray!! 🎉

@DKoctopus
Copy link

DKoctopus commented Mar 2, 2018

@jaywilliams @vrann Was having the same issue, tried this 69a0f5c

It fixed for canda but having similar issues for united kingdom, any thoughts?

@jaywilliams
Copy link
Contributor Author

@DKoctopus We haven't noticed anything with the UK, but then again, we don't get many orders there.

magento-devops-reposync-svc pushed a commit that referenced this pull request Jan 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants