-
Notifications
You must be signed in to change notification settings - Fork 15
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
Add Honduras Localization Pack #6
base: master
Are you sure you want to change the base?
Conversation
Honduras Localization Pack
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Julievrz I don't know if you can check it about translations.
@PrestaShop/product-team Could you validate unit information please?
It looks ok for the language and the ISO code. The Spanish (HN) project is not available on Crowdin at the moment but we will be able to add it later on if translators are interested in working on it. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @vienafromm and @Julievrz I'm afraid the following informations in the PR are not helpful 😅
BC breaks? | yes |
---|---|
Deprecations? | yes |
Fixed ticket? | Fixes |
How to test? | Test it |
How do we test this works?
Is there deprecations? If yes, what are they?
Is there BC breaks? If yes, what are they?
Without these informations I dont see how we can validate and merge this PR
@matks are these questions and form required for the core project, does it fit here ? Localization file can be updated by non-tech people I guess. So BC and Deprecations can be tough questions. |
We can remove them indeed, but then why say "Yes" 😅 because now I'm afraid. I just want to understand the answer to these. If the answer is "I did not know so I wrote Yes" it's OK 😉 but I need to be informed |
@matks Correct, I did not know so I wrote Yes. I don't know what BC and Deprecations are. :S |
Could you take a look at this PR? It seems it has been approved and it is ready to be merged. (I would like to add <states> tag to this file). Thank you, have a great day |
It's ok. As I said a while ago, it looks ok for the language and the ISO code: at least for the currency ISO code which is HNL. For the language ISO code, it's HND but the Spanish (HN) project is not available on Crowdin at the moment. We'll still be able to add it later on if translators are interested in working on it. Information about the metric system seems ok as well. |
@Progi1984 ping! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @vienafromm,
I tried to install the honduras
pack.
But I found that the currency symbol = L
I checked the Product page > Pricing Tab => I have different symbols
@Julievrz what do you think? what is the exact symbol should be displayed?
Thanks!
It seems that both are acceptable. HNL is the ISO 4217. Maybe @rickygzz has more insights? |
Sorry, it was my mistake. Yes, the correct currency is HNL (It means Honduran Lempira) Thanks for following up on this! <3 Hugs, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change approved!
Thanks @vienafromm, let's go with "HNL" then. :-) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hi @vienafromm,
I still have the same issues with PS1785 & the develop branch.
I tried these steps:
- Install PS1785 with English language & France country
- Add the new file
hn.xml
in the pathPrestaShop/localization/
- Go to BO > International > Localization page > Import the Honduras pack
- Go to BO > International > Localization > Currencies page
- See error, the symbol is incorrect
- Go to FO > Select the new currency
- See error
- Go to BO > Catalog > Product page > edit a product > Pricing tab
- See error
Did I miss something?
Thanks!
Hello, It seems HNL and L are both ok. The first one is the currency ISO 4217 code (like USD or EUR) and the second one is the sign (like $ or €) as stated in their country laws. So in my opinion, the xml file currency attributes should be iso_code="HNL" sign="L" (instead of HNL), as follows:
However, I was intrigued why it didn't follow the proposed XML file during the QA. It seems PS pulls its data from the standardized CLDR database. The currency data for Honduran Lempira is pulled from this file PrestaShop/PrestaShop/localization/CLDR/core/common/main/es.xml line 6461. That being said, it seems anytime soon we should clean this PrestaShop/LocalizationFiles repository and remove the currency tags, since it seems its not being used anymore. Hope it helps, Source:The ISO three-letter alphabetic code for its currency is HNL (see ISO 4217 https://www.iso.org/iso-4217-currency-codes.html which refers to https://www.six-group.com/en/products-services/financial-information/data-standards.html and it offers a list updated up to April 1st 2022) The currency symbol (sign) is L. This as stated in Wikipedia and in the Law Ley Monetaria Decreto - No. 51-1950 in its first article
|
You are completely right. The problem is that older versions of PrestaShop retrieve localization files from the same source as newer versions. We need to figure out a way to fix that. |
Honduras Localization Pack