-
-
Notifications
You must be signed in to change notification settings - Fork 8.4k
🌐 Add Portuguese translation for docs/pt/docs/advanced/middleware.md
#12704
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
🌐 Add Portuguese translation for docs/pt/docs/advanced/middleware.md
#12704
Conversation
|
📝 Docs preview for commit 759f805 at: https://cd4af8cb.fastapitiangolo.pages.dev Modified Pages |
docs/pt/docs/advanced/middleware.md
There was a problem hiding this comment.
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.
|
@ceb10n and @cassiobotaro, could one of you do the kindness of reviewing this pull request? |
There was a problem hiding this comment.
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
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" |
There was a problem hiding this comment.
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
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>. |
There was a problem hiding this comment.
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 😄
There was a problem hiding this comment.
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!
|
Hey @ceb10n and @nazarepiedady thank you for your feedback! I'll adress the changes and make a PR |
|
📝 Docs preview for commit ba72208 at: https://8d94dad3.fastapitiangolo.pages.dev Modified Pages |
|
I just committed the changes. @ceb10n if you need anything else, feel free to ask! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
|
Great, thanks! @devluisrodrigues 🎉 |
No description provided.