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

[5.0] Error Pages language #41934

Merged
merged 2 commits into from
Sep 30, 2023
Merged

Conversation

brianteeman
Copy link
Contributor

Added a note to the strings in the installation language file that is used for building the error page.

As seen in the screenshot several language packs have translated instead of localised the string or as in the case of German does not include the country part of the code resulting in unclear options

image

cc @tecpromotion

Link to documentations

Please select:

  • Documentation link for docs.joomla.org:

  • No documentation changes for docs.joomla.org needed

  • Pull Request link for manual.joomla.org:

  • No documentation changes for manual.joomla.org needed

Added a note to the strings in the installation language file that is used for building the error page.

As seen in the screenshot several language packs have translated instead of localised the string or as in the case of German does not include the country part of the code resulting in unclear options

@tecpromotion
@joomla-cms-bot joomla-cms-bot added Language Change This is for Translators PR-5.0-dev labels Sep 26, 2023
@ceford
Copy link
Contributor

ceford commented Sep 27, 2023

I think this is OK. However, I noticed that line 164 has this:

INSTL_DEFAULTLANGUAGE_NATIVE_LANGUAGE_NAME="English (UK)" ; IMPORTANT NOTE FOR TRANSLATORS: Do not literally translate this line, instead add the localised name of the language. For example Spanish will be Español

Should that have (ES) on the end? Or for consistency use French FR

Went over to Crowdin for the first time for a year or so to see the comment in context.


This comment was created with the J!Tracker Application at issues.joomla.org/tracker/joomla-cms/41934.

@brianteeman
Copy link
Contributor Author

That is unrelated to this pr

@brianteeman
Copy link
Contributor Author

updated branch to retrigger drone

@HLeithner HLeithner merged commit 1d137d5 into joomla:5.0-dev Sep 30, 2023
3 checks passed
@brianteeman
Copy link
Contributor Author

thanks

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Language Change This is for Translators
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants