8000 :globe_with_meridians: Add Portuguese translation for `docs/pt/docs/advanced/templates.md` by SamuelBFavarin · Pull Request #11338 · fastapi/fastapi · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@SamuelBFavarin
Copy link
Contributor
@SamuelBFavarin SamuelBFavarin commented Mar 23, 2024

Added Portuguese translation to templates.md

@tiangolo
Copy link
Member

📝 Docs preview for commit dfb475e at: https://d1c93045.fastapitiangolo.pages.dev

Copy link
@codespearhead codespearhead left a comment
8000

Choose a reason for hiding this comment

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

  • Points out typos.
  • Translates excerpts that are yet to be translated.
  • Changes expressions that don't sound smooth in Brazilian Portuguese.

Copy link
@gustavoprezoto gustavoprezoto left a comment

Choose a reason for hiding this comment

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

just left a few small suggestions (:

@alejsdev alejsdev changed the title Added Portuguese 🇧🇷 translation to templates.md 🌐 Add Portuguese translation for docs/pt/docs/advanced/templates.md Mar 24, 2024
SamuelBFavarin and others added 3 commits March 25, 2024 09:22
Co-authored-by: Gustavo <62812585+gustavoprezoto@users.noreply.github.com>
Co-authored-by: Gustavo <62812585+gustavoprezoto@users.noreply.github.com>
Co-authored-by: Gustavo <62812585+gustavoprezoto@users.noreply.github.com>
@tiangolo
Copy link
Member

📝 Docs preview for commit 5749091 at: https://06d83cff.fastapitiangolo.pages.dev

@tiangolo
Copy link
Member

📝 Docs preview for commit 42d116c at: https://9347d220.fastapitiangolo.pages.dev

@SamuelBFavarin
Copy link
Contributor Author

Thanks for your review @codespearhead and @gustavoprezoto! Could you check my updates?

@alejsdev alejsdev added lang-all Translations lang-pt Portuguese translations awaiting-review labels Mar 25, 2024
Copy link
@EliasOlie EliasOlie left a comment

Choose a reason for hiding this comment

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

Acredito que ao ajustar de acordo com essa sugestão: https://github.com/tiangolo/fastapi/pull/11338/files#r1537541637 ficará ainda mais excelente!

@tiangolo
Copy link
Member

📝 Docs preview for commit bb455a2 at: https://ad30907a.fastapitiangolo.pages.dev

@tiangolo
Copy link
Member

📝 Docs preview for commit 9f91742 at: https://a0b2b9f3.fastapitiangolo.pages.dev

Copy link
@codespearhead codespearhead left a comment

Choose a reason for hiding this comment

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

Ready to merge @alejsdev

@alejsdev alejsdev merged commit ff41af8 into fastapi:master Mar 28, 2024
@alejsdev
Copy link
Member

Nice! Thanks for the contribution @samuel-favarin-dbc 🥳

And thanks for the reviews 😄 @gustavoprezoto @EliasOlie @codespearhead

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.

7 participants

0