Skip to content

馃寪 Add Portuguese translation for docs/pt/docs/advanced/middleware.md - #12704

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

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

Conversation

@lasr3

@lasr3 lasr3 commented Oct 28, 2024

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!

@lasr3

lasr3 commented Oct 29, 2024

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

@lasr3
lasr3 requested a review from ceb10n October 29, 2024 11:51
@lasr3

lasr3 commented Oct 29, 2024

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