Skip to content

🌐 Add German translations for recent new pages (using LLMs)#13929

Closed
tiangolo wants to merge 5 commits intomasterfrom
translations-de
Closed

🌐 Add German translations for recent new pages (using LLMs)#13929
tiangolo wants to merge 5 commits intomasterfrom
translations-de

Conversation

@tiangolo
Copy link
Member

@tiangolo tiangolo commented Jul 26, 2025

🌐 Add German translations for recent new pages (using LLMs)

While reviewing these files, instead of suggesting specific changes and tweaks, it's better to tweak the LLM prompt (#13931) to make the LLM generate better translations.

@github-actions github-actions bot added the lang-all Translations label Jul 26, 2025
@github-actions
Copy link
Contributor

📝 Docs preview for commit 602f8c2 at: https://1d30ac73.fastapitiangolo.pages.dev

Modified Pages

@tiangolo tiangolo marked this pull request as ready for review July 26, 2025 20:46
@tiangolo tiangolo added awaiting-review lang-de German translations labels Jul 26, 2025
nilslindemann pushed a commit to nilslindemann/fastapi that referenced this pull request Aug 4, 2025
nilslindemann added a commit to nilslindemann/fastapi that referenced this pull request Aug 4, 2025
There are a few more changes here than necessary, because I created these translations from scratch without an existing translation and without wordlists in the German LLM, to learn which words the AI actually needs to have defined (less than expected). I then was too lazy to remove the not super necessary changes, :-D. But The translations are good, I manually checked them.

You may apply this commit on fastapi#13929. Regenerating it is not necessary.
nilslindemann and others added 2 commits August 4, 2025 17:13
There are a few more changes here than necessary, because I created these translations from scratch without an existing translation and without wordlists in the German LLM, to learn which words the AI actually needs to have defined (less than expected). I then was too lazy to remove the not super necessary changes, :-D. But The translations are good, I manually checked them.

You may apply this commit on #13929. Regenerating it is not necessary.
@github-actions
Copy link
Contributor

github-actions bot commented Aug 4, 2025

📝 Docs preview for commit dda4005 at: https://7ddfcfbd.fastapitiangolo.pages.dev

Modified Pages

Copy link
Contributor

@nilslindemann nilslindemann left a comment

Choose a reason for hiding this comment

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

Good 👍

nilslindemann added a commit to nilslindemann/fastapi that referenced this pull request Aug 5, 2025
Like the previous commit, but for fastapi#13929 (translations-de). Please apply it there.
Copy link
Contributor

@nilslindemann nilslindemann left a comment

Choose a reason for hiding this comment

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

This is good 👍 and can be merged as is, if you want, I will add the permalinks in a later PR.

@nilslindemann
Copy link
Contributor

#13931, #13929 (this one), and #13930 can be closed, as I have applied them in #14015 and their changes will be committed when that one is committed. Also, I have resolved merge conflicts there already.

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

Labels

approved-1 lang-all Translations lang-de German translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants