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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃寪 Start Dutch (Netherlands) translations #4346

Closed
wants to merge 3 commits into from
Closed

馃寪 Start Dutch (Netherlands) translations #4346

wants to merge 3 commits into from

Conversation

ghost
Copy link

@ghost ghost commented Dec 29, 2021

After the success of #3941, here a start for Dutch translations of FastAPI. If this PR is approved and merged I will start adding the translations 馃槃!

@Kludex
Copy link
Sponsor Collaborator

Kludex commented Dec 31, 2021

This comment was a thing, but I'm not sure if it still is: #1497 (comment)

Just mentioning, don't mind me :)

@tiangolo
Copy link
Owner

Cool @adriaanmolendijk!

As @Kludex says, the biggest advantage with translations is when people don't speak English already and speak mostly that language.

But still, if you are willing to add translations, let's do it! 馃槑 馃崻

I created #4701 to track the Dutch translations and I added the setup there so that new translations to be reviewed are notified in that issue.

Thank you!

@tiangolo tiangolo changed the title Starting dutch (Netherlands) translations 馃寪 Start Dutch (Netherlands) translations Mar 18, 2022
@tiangolo
Copy link
Owner

I see that I can't push to your branch to trigger CI, so I created a new PR with this here: #4703

Thanks for your work and contribution! I'm gonna close this one now.

@tiangolo tiangolo closed this Mar 18, 2022
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.

None yet

3 participants