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

Fixes U4 6510 - Norwegian labels, dictionary and language #933

Closed
wants to merge 4 commits into from

Conversation

engern
Copy link
Contributor

@engern engern commented Nov 29, 2015

I'm not sure how localization works in Umbraco, but it seems to be a comparing of the first 2 letters of the culture name and the filname, because nothing is translated when you switch to Norwegian
language. When changing the filename from no.xml to nb.xml, translations are applied

It seems to be a check on the first 2 letters of the culture name and
the filname, because nothing is translated when you switch to norwegian
language. When changing the filename from no.xml to nb.xml, translations
are applied
@nul800sebastiaan
Copy link
Member

Wow, I don't think this ever worked correctly! Thanks!

@engern
Copy link
Contributor Author

engern commented Dec 16, 2015

I guess this merge failed because I applied the renaming before the changes in #935 ?

@nul800sebastiaan
Copy link
Member

It didn't fail! :-)

@engern
Copy link
Contributor Author

engern commented Dec 23, 2015

Ok, the pull request just didn't close like they usually does, and it says "This branch has conflicts that must be resolved" I also downloaded 7.4 BETA today and it seems the language file is still called "no.xml"

@nul800sebastiaan
Copy link
Member

Weird, looks like I messed up then! Will investigate after the holidays, thanks for letting me know!

@nul800sebastiaan
Copy link
Member

Alright, fixed this all up now and 7.3.5+ will have the new translations and the nb.xml file by default, thanks again!

@engern
Copy link
Contributor Author

engern commented Jan 7, 2016

Perfect 👍 Any chance #932 could be considered for 7.3.5?

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

Successfully merging this pull request may close these issues.

2 participants