Skip to content

Added Puerto Rico to the Directory module's install data script. #6191

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

Closed
wants to merge 1 commit into from
Closed

Added Puerto Rico to the Directory module's install data script. #6191

wants to merge 1 commit into from

Conversation

mathewhellums
Copy link

@mathewhellums mathewhellums commented Aug 17, 2016

I recently came across an issue with UPS shipping rates not being populated when shipping to a Puerto Rican address. I had set up UPS with my shipping account info and shipper id and enabled all available shipping methods, but UPS would not return any shipping rates when using a Puerto Rico zip code/address. I opened a ticket with Enterprise Support (Ticket #03187690), and it is currently in an "escalated" state.

After digging a bit, I discovered the ship carrier was looking for Puerto Rico in the database but it does not exist. Apparently it did exist in Magento 1 but looks to have not been included in Magento 2. Adding the row for Puerto Rico to the database immediately solved UPS not returning shipping rates for Puerto Rico.

This update adds Puerto Rico back to the install data script.

@magento-cicd2
Copy link
Contributor

magento-cicd2 commented Aug 17, 2016

CLA assistant check
All committers have signed the CLA.

@vrann vrann self-assigned this Mar 20, 2017
@vrann vrann added this to the March 2017 milestone Mar 20, 2017
@vrann
Copy link
Contributor

vrann commented Mar 20, 2017

@mathewhellums The branch for this PR was deleted. Can you please re-create the PR from the new branch?

@vrann vrann closed this Mar 20, 2017
magento-engcom-team pushed a commit that referenced this pull request Oct 4, 2020
[honey] MC-37582: Fix Failing WebAPI GraphQL tests when DB table prefixes are enabled
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.

4 participants