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

Made Dutch (NL) localization a neutral culture, not just for Belgium #109

Merged
merged 2 commits into from
Mar 28, 2014

Conversation

ErikSchierboom
Copy link
Contributor

At the moment, there is a Dutch localization file but it is targeted specifically to Belgian Dutch. However, all the localized strings are also valid Netherlands Dutch. So I have renamed the Resources.nl-BE.resx file to the more general Resources.nl.resx.

@faisalr
Copy link
Contributor

faisalr commented Mar 27, 2014

Good addition, this is called a neutral culture.

@ErikSchierboom ErikSchierboom changed the title Made Dutch (NL) localization generic, not just for Belgium Made Dutch (NL) localization a neutral culture, not just for Belgium Mar 27, 2014
@MehdiK
Copy link
Member

MehdiK commented Mar 28, 2014

Cool. Thanks for this.

I don't know how this got in without tests in the first place. So I am going to ask you to kindly add some unit tests for it first!! To implement unit tests you might check out unit tests for one of the other cultures.

@ErikSchierboom
Copy link
Contributor Author

@MehdiK But of course. I have added the tests.

@MehdiK
Copy link
Member

MehdiK commented Mar 28, 2014

I love it that you also added TimeSpan resources :)

MehdiK added a commit that referenced this pull request Mar 28, 2014
Made Dutch (NL) localization a neutral culture, not just for Belgium
@MehdiK MehdiK merged commit cd28253 into Humanizr:master Mar 28, 2014
@ErikSchierboom ErikSchierboom deleted the dutch-localization branch March 28, 2014 13:58
MehdiK added a commit that referenced this pull request Mar 28, 2014
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.

3 participants