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 Dutch translations (in intl-icu format) #60

Merged
merged 1 commit into from
Mar 29, 2021

Conversation

evertharmeling
Copy link
Contributor

Added Dutch translations. Because of the somewhat complex nature of the Dutch language (0 vs 1 vs many) I've chosen to use the intl-icu format as it (suppose to be/) is the new standard and has better support for the complex scenarios. And as the minimum requirements of the bundle are symfony/*:^4.4 it should not be a problem. Also Christophe mentioned to it in #56 (comment).

If we would refactor all translations in one go to the icu-intl format and first use the old notation, let me know I would adjust this PR.

As for the Dutchies who want to check this PR, I've used the 'rules' in https://taaladvies.net/taal/advies/vraag/511 to translate it (used in \SymfonyCasts\Bundle\VerifyEmail\Model\VerifyEmailSignatureComponents::getExpirationMessageKey()), see:

Uitzonderingen op de algemene regel vormen de zelfstandige naamwoorden kwartier, uur en jaar. Die hebben in de genoemde combinaties gewoonlijk de enkelvoudsvorm.

@weaverryan weaverryan merged commit 2e4b943 into SymfonyCasts:master Mar 29, 2021
@weaverryan
Copy link
Contributor

Thanks @evertharmeling! Unless there is some problem with mixing the translation formats that i'm not aware of, I'm ok with having either :).

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