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

Skip to content

🌐 Add Portuguese translation for docs/pt/docs/advanced/middleware.md#12704

Merged
alejsdev merged 4 commits into
fastapi:masterfrom
devluisrodrigues:PortugueseTranslation
Oct 30, 2024
Merged

🌐 Add Portuguese translation for docs/pt/docs/advanced/middleware.md#12704
alejsdev merged 4 commits into
fastapi:masterfrom
devluisrodrigues:PortugueseTranslation

Conversation

@devluisrodrigues

Copy link
Copy Markdown
Contributor

No description provided.

@github-actions github-actions Bot added the lang-all Translations label Oct 28, 2024
@github-actions

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev changed the title 🌐 Add Portuguese translation for docs/pt/docs/advanced/middleware.md 🌐 Add Portuguese translation for docs/pt/docs/advanced/middleware.md Oct 28, 2024
@alejsdev alejsdev added awaiting-review lang-pt Portuguese translations labels Oct 28, 2024

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

Excellent, keep going with this kind of contribution.

@nazarepiedady

Copy link
Copy Markdown
Contributor

@ceb10n and @cassiobotaro, could one of you do the kindness of reviewing this pull request?

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

nice work @devluisrodrigues

Just a line that you've forgotten to remove, and a quote in the admonition

Comment thread docs/pt/docs/advanced/middleware.md Outdated

**FastAPI** inclui vários middlewares para casos de uso comuns, veremos a seguir como usá-los.

/// note | "Detalhes Técnicos"

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.

You can remove the quotes here:

/// note | Detalhes Técnicos

Comment thread docs/pt/docs/advanced/middleware.md Outdated
* <a href="https://github.com/encode/uvicorn/blob/master/uvicorn/middleware/proxy_headers.py" class="external-link" target="_blank">Uvicorn's `ProxyHeadersMiddleware`</a>
* <a href="https://github.com/florimondmanca/msgpack-asgi" class="external-link" target="_blank">MessagePack</a>

To see other available middlewares check <a href="https://www.starlette.io/middleware/" class="external-link" target="_blank">Starlette's Middleware docs</a> and the <a href="https://github.com/florimondmanca/awesome-asgi" class="external-link" target="_blank">ASGI Awesome List</a>.

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.

You've forgotten to remove this line 😄

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

oh haha, I'm sorry for that. I'll fix it!

@devluisrodrigues

Copy link
Copy Markdown
Contributor Author

Hey @ceb10n and @nazarepiedady thank you for your feedback! I'll adress the changes and make a PR

@github-actions

Copy link
Copy Markdown
Contributor

@devluisrodrigues

Copy link
Copy Markdown
Contributor Author

I just committed the changes. @ceb10n if you need anything else, feel free to ask!

@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 enabled auto-merge (squash) October 30, 2024 19:57
@alejsdev

Copy link
Copy Markdown
Member

Great, thanks! @devluisrodrigues 🎉

@alejsdev alejsdev merged commit 3184b5c into fastapi:master Oct 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

approved-2 lang-all Translations lang-pt Portuguese translations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants