Thanks to visit codestin.com
Credit goes to github.com

Skip to content

🌐 Add Portuguese translation for docs/pt/docs/advanced/settings.md#11739

Merged
alejsdev merged 3 commits into
fastapi:masterfrom
Joao-Pedro-P-Holanda:docs/advanced-settings-pt
Jun 20, 2024
Merged

🌐 Add Portuguese translation for docs/pt/docs/advanced/settings.md#11739
alejsdev merged 3 commits into
fastapi:masterfrom
Joao-Pedro-P-Holanda:docs/advanced-settings-pt

Conversation

@Joao-Pedro-P-Holanda

Copy link
Copy Markdown
Contributor

Adding portuguese translation for docs/pt/docs/advanced/settings.md

@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit e244776 at: https://dbae242b.fastapitiangolo.pages.dev

@alejsdev alejsdev changed the title 🌐 Add Portuguese translation for docs/pt/docs/advanced/settings.md 🌐 Add Portuguese translation for docs/pt/docs/advanced/settings.md Jun 20, 2024
@alejsdev alejsdev added lang-all Translations lang-pt Portuguese translations awaiting-review labels Jun 20, 2024

@ceb10n ceb10n left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Very good translation, congrats! ;)

I added some suggestions, but it's a good translation whether you accept the suggestions or not.

Thank you

Comment thread docs/pt/docs/advanced/settings.md Outdated
Comment thread docs/pt/docs/advanced/settings.md Outdated
@Joao-Pedro-P-Holanda

Copy link
Copy Markdown
Contributor Author

@ceb10n thanks for the review, i would like to address these suggestions in a new commit before merging.

changing "reutilizados de novo e de novo para cada chamada da função" in line 420 to "sempre reutilizados para cada nova chamada da função"

restructuring phrase in line 308, and using the sentence "sobrescrevendo a dependência" instead of "sobrescrita de dependência"
@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit b808dfc at: https://3bc1dbff.fastapitiangolo.pages.dev

@alejsdev alejsdev merged commit 85bad33 into fastapi:master Jun 20, 2024
@alejsdev

Copy link
Copy Markdown
Member

Cool! Thanks to both of you for the help 🤓 @Joao-Pedro-P-Holanda @ceb10n

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-pt Portuguese translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants