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

Added states to ar.xml according to ISO 3166-2. #16

Merged
merged 1 commit into from
May 11, 2022

Conversation

rickygzz
Copy link
Contributor

@rickygzz rickygzz commented Dec 19, 2021

Questions Answers
Description? Please read below.
Fixed ticket? Fixes #14 (part of it, more PR's are coming :)

Added <states> tag according to ISO 3166-2 (https://www.iso.org/obp/ui/#iso:code:3166:AR).

Copy link
Contributor

@Julievrz Julievrz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All ISO codes in this file match the ones on https://github.com/PrestaShop/PrestaShop/blob/develop/install-dev/data/xml/state.xml. I only have a doubt regarding "Ciudad de Buenos Aires". I left a comment.

@rickygzz rickygzz force-pushed the rickygzz-patch-ar-xml branch from 0cadfe7 to c61b8a5 Compare December 26, 2021 20:28
Copy link
Member

@eternoendless eternoendless left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@sLorenzini sLorenzini self-assigned this May 10, 2022
Copy link

@sLorenzini sLorenzini left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hello @rickygzz,

Tested on develop and 178x branches
7.2.21 PHP version

  • Add fix on ar.xml file in mystore folder > localization
  • Go to locations > states > filter by country "Argentina"
  • Deleted all states of Argentina
  • Go to Localization > Select Argentina
  • Choose "states" in content to import
  • Set to "No" the "Download pack data" switch button (if set to yes not working, the states list of Argentina doesn't exist on prestashop.com)
  • Click on Import button
  • Go back to locations > states and update the page, all states on the list are displayed

QA approved ✅

Thank you

@sLorenzini sLorenzini added QA ✔️ Status: check done, code approved and removed waiting for QA labels May 10, 2022
@Progi1984 Progi1984 merged commit 06f1533 into PrestaShop:master May 11, 2022
@Progi1984
Copy link
Member

Thanks @rickygzz & @sLorenzini

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
QA ✔️ Status: check done, code approved Wording ✅
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add states to North America, Central America and South America
5 participants