Skip to content

🌐 Add Dutch translation for docs/nl/docs/index.md#5366

Closed
maxscheijen wants to merge 2 commits intofastapi:masterfrom
maxscheijen:add-nl-docs-index
Closed

🌐 Add Dutch translation for docs/nl/docs/index.md#5366
maxscheijen wants to merge 2 commits intofastapi:masterfrom
maxscheijen:add-nl-docs-index

Conversation

@maxscheijen
Copy link
Copy Markdown
Contributor

Relates to #4701

This translatesdocs/index.md into Dutch.

@codecov
Copy link
Copy Markdown

codecov bot commented Sep 9, 2022

Codecov Report

Merging #5366 (7c42fce) into master (6620273) will not change coverage.
The diff coverage is n/a.

@@            Coverage Diff            @@
##            master     #5366   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          540       540           
  Lines        13936     13934    -2     
=========================================
- Hits         13936     13934    -2     
Impacted Files Coverage Δ
tests/utils.py 100.00% <0.00%> (ø)
fastapi/concurrency.py 100.00% <0.00%> (ø)
fastapi/applications.py 100.00% <0.00%> (ø)
fastapi/dependencies/utils.py 100.00% <0.00%> (ø)
fastapi/exception_handlers.py 100.00% <0.00%> (ø)
tests/test_starlette_exception.py 100.00% <0.00%> (ø)
tests/test_union_inherited_body.py 100.00% <0.00%> (ø)
..._sql_databases_peewee/test_sql_databases_peewee.py 100.00% <0.00%> (ø)

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Sep 9, 2022

📝 Docs preview for commit afc1711 at: https://631b38ee7c132600636d31c3--fastapi.netlify.app

@tiangolo tiangolo added lang-all Translations awaiting-review lang-nl Dutch translations labels Sep 9, 2022
@github-actions github-actions bot mentioned this pull request Sep 9, 2022
@github-actions
Copy link
Copy Markdown
Contributor

📝 Docs preview for commit 7c42fce at: https://632430f641b171064623fa0a--fastapi.netlify.app

@Kludex
Copy link
Copy Markdown
Member

Kludex commented Sep 17, 2022

I guess it's duplicated of #4770 😟

A review there will probably be welcome 👀

Copy link
Copy Markdown

@tyzh-dev tyzh-dev left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A couple typos and inconsistency with combining informal and formal writing.

Terugkomend op het vorige code voorbeeld, **FastAPI** zal:

* Valideren dat er een `item_id` bestaat in het pad voor `GET` en `PUT` verzoeken.
* Valideren dat `item_id` van het type `int` is voor `GET` en `PUT` verzoeken.
Copy link
Copy Markdown

@tyzh-dev tyzh-dev Apr 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Valideren dat de item_id

## Prestaties

Independent TechEmpower benchmarks show **FastAPI** applications running under Uvicorn as <a href="https://www.techempower.com/benchmarks/#section=test&runid=7464e520-0dc2-473d-bd34-dbdfd7e85911&hw=ph&test=query&l=zijzen-7" class="external-link" target="_blank">one of the fastest Python frameworks available</a>, only below Starlette and Uvicorn themselves (used internally by FastAPI). (*)
nafhankelijke TechEmpower benchmarks tonen **FastAPI** applicaties draaiend onder Uvicorn als <a href="https://www.techempower.com/benchmarks/#section=test&runid=7464e520-0dc2-473d-bd34-dbdfd7e85911&hw=ph&test=query&l=zijzen-7" class="external-link" target="_blank">een van de snelste Python frameworks beschikbaar</a>, alleen onder Starlette en Uvicorn zelf (intern gebruikt door FastAPI). (*)
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do you mean Onafhankelijke instead?

* <a href="https://pyyaml.org/wiki/PyYAMLDocumentation" target="_blank"><code>pyyaml</code></a> - Required for Starlette's `SchemaGenerator` support (you probably don't need it with FastAPI).
* <a href="https://requests.readthedocs.io" target="_blank"><code>requests</code></a> - Vereist als je de `TestClient` wilt gebruiken.
* <a href="https://jinja.palletsprojects.com" target="_blank"><code>jinja2</code></a> - Vereist als u de standaard templateconfiguratie wilt gebruiken.
* <a href="https://andrew-d.github.io/python-multipart/" target="_blank"><code>python-multipart</code></a> - Vereist als je het <abbr title="chet omzetten van de string die uit een HTTP verzoek komt in Python data">"parsen"</abbr> van formulieren wil ondersteunen met `requests.form()`.
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Change chet omzetten to het omzetten van

* <a href="https://github.com/ijl/orjson" target="_blank"><code>orjson</code></a> - Vereist als je `ORJSONResponse` wilt gebruiken.

You can install all of these with `pip install "fastapi[all]"`.
U kunt deze allemaal installeren met `pip install "fastapi[all]"`.
Copy link
Copy Markdown

@tyzh-dev tyzh-dev Apr 30, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You are mixing je and u in the whole markdown file. Isn't it better to either choose the formal or informal version for consistency?

@tyzh-dev
Copy link
Copy Markdown

tyzh-dev commented Apr 30, 2023

@Kludex

I guess it's duplicated of #4770 😟

A review there will probably be welcome 👀

That one translates the tutorial instead of the index page. So, not a duplicate but yeah I can take a look.

@tiangolo tiangolo changed the title 🌐 Add Dutch translation for docs/nl/docs/index.md 🌐 Add Dutch translation for docs/nl/docs/index.md Jun 27, 2023
@tiangolo
Copy link
Copy Markdown
Member

It seems this was handled in another PR, so I'm gonna close this one, but thanks for the effort! ☕

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review lang-all Translations lang-nl Dutch translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants