🌐 Add Portuguese translation for docs/pt/docs/advanced/openapi-callbacks.md#12705
Conversation
|
📝 Docs preview for commit 4570a7a at: https://d7815a6d.fastapitiangolo.pages.dev Modified Pages |
docs/pt/docs/how-to/openapi-callbacks.md
|
@devfernandoa, normally, it is recommended to only modify one single file per pull request, but I think in this one, we could benefit from an exception. Keep following this path of collaborating with open-source initiatives. |
|
@ceb10n and @cassiobotaro, could you do the kindness of reviewing this pull request? |
|
@devfernandoa please separate this in 2 PRs, one for each file, this way it will be easier to review. 🤓☕ @nazarepiedady I'll ask you to please don't give instructions about managing the PRs or the repo, those things would be decided by someone in the FastAPI team, and getting external instructions could be confusing for people contributing. Also there's no need to ping others. Thanks! |
|
Sorry guys, I forgot to remove the file when switching branches. |
|
📝 Docs preview for commit 948b71e at: https://a27475ef.fastapitiangolo.pages.dev Modified Pages |
ceb10n
left a comment
There was a problem hiding this comment.
Great work @devfernandoa ✨
Just some lines that need to be removed, and some minor suggestions 😄
|
|
||
| * Enviar uma solicitação de pagamento para o desenvolvedor externo. | ||
| * Coletar o dinheiro. | ||
| * Send a notification back to the API user (the external developer). |
There was a problem hiding this comment.
You've forgotten to remove this line 😄
| {!../../docs_src/openapi_callbacks/tutorial001.py!} | ||
| ``` | ||
|
|
||
| /// tip | dica |
There was a problem hiding this comment.
Just a silly suggestion:
We alway start with a capital letter the translation of the admonitions
|
|
||
| /// | ||
|
|
||
| The only new thing is the `callbacks=invoices_callback_router.routes` as an argument to the *path operation decorator*. We'll see what that is next. |
|
|
||
| Esse exemplo não implementa o callback em si (que poderia ser apenas uma linha de código), apenas a parte da documentação. | ||
|
|
||
| /// tip | dica |
There was a problem hiding this comment.
here to, just start with a capital letter if you want :D
|
|
||
| O callback real é apenas uma solicitação HTTP. | ||
|
|
||
| When implementing the callback yourself, you could use something like <a href="https://www.python-httpx.org" class="external-link" target="_blank">HTTPX</a> or <a href="https://requests.readthedocs.io/" class="external-link" target="_blank">Requests</a>. |
There was a problem hiding this comment.
This line needs to be removed to
|
|
||
| Então vamos usar esse mesmo conhecimento para documentar como a *API externa* deveria ser... criando as *operações de rota* que a *API externa* deveria implementar (as que sua API irá chamar). | ||
|
|
||
| /// tip | dica |
There was a problem hiding this comment.
here to, just start with a capital letter if you want :D
| } | ||
| ``` | ||
|
|
||
| /// tip | dica |
There was a problem hiding this comment.
here to, just start with a capital letter if you want :D
| {!../../docs_src/openapi_callbacks/tutorial001.py!} | ||
| ``` | ||
|
|
||
| /// tip | dica |
There was a problem hiding this comment.
here to, just start with a capital letter if you want :D
|
Not a so important thing that I've noticed: The title is incorrect: this file lives under |
docs/pt/docs/how-to/openapi-callbacks.mddocs/pt/docs/advanced/openapi-callbacks.md
|
Just fixed these changes, thanks for the review @ceb10n ! |
|
📝 Docs preview for commit ee137b7 at: https://7fc0c98e.fastapitiangolo.pages.dev Modified Pages |
|
Thank you for being so interested in contributing to FastAPI! @devfernandoa 🤓 |
No description provided.