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

Skip to content

🌐 Add Portuguese translation for docs/pt/docs/tutorial/dependencies/dependencies-with-yield.md#11848

Merged
alejsdev merged 3 commits into
fastapi:masterfrom
Joao-Pedro-P-Holanda:docs/dependencies-dep-with-yield-pt
Jul 18, 2024
Merged

🌐 Add Portuguese translation for docs/pt/docs/tutorial/dependencies/dependencies-with-yield.md#11848
alejsdev merged 3 commits into
fastapi:masterfrom
Joao-Pedro-P-Holanda:docs/dependencies-dep-with-yield-pt

Conversation

@Joao-Pedro-P-Holanda

Copy link
Copy Markdown
Contributor

No description provided.

@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit ac056ff at: https://c0763452.fastapitiangolo.pages.dev

@alejsdev alejsdev added lang-all Translations lang-pt Portuguese translations awaiting-review labels Jul 15, 2024

@lucasbalieiro lucasbalieiro 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.

Seems like a nice and well-crafted translation.

Good job, @Joao-Pedro-P-Holanda !

Just a few small things that I’ve noticed

Comment thread docs/pt/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/pt/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
Comment thread docs/pt/docs/tutorial/dependencies/dependencies-with-yield.md Outdated

@nazarepiedady nazarepiedady 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.

Beyond the little error mentioned by @lucasbalieiro, there is nothing to complain about, and this translation can be merged and then updated missing point.

@Joao-Pedro-P-Holanda

Copy link
Copy Markdown
Contributor Author

Before merging i would like to address the points marked by @lucasbalieiro

@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.

Great translation @Joao-Pedro-P-Holanda 🚀🚀🚀

I only saw one minor typo in addition to the revisions already made.

Thank you!

Comment thread docs/pt/docs/tutorial/dependencies/dependencies-with-yield.md Outdated
@tiangolo

Copy link
Copy Markdown
Member

📝 Docs preview for commit 795e7a2 at: https://612eb8ab.fastapitiangolo.pages.dev

@lucasbalieiro lucasbalieiro 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.

Nice!!!!

@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.

🚀🚀🚀

@alejsdev alejsdev merged commit a5fbbee into fastapi:master Jul 18, 2024
@alejsdev

Copy link
Copy Markdown
Member

Cool! Thanks to all for the contribution and help 🚀 🤓

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.

8 participants