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

Skip to content

🌐 Add Portuguese translation for docs/pt/docs/advanced/dataclasses.md#12475

Merged
alejsdev merged 3 commits into
fastapi:masterfrom
leoscarlato:dataclass-translation-pt
Oct 22, 2024
Merged

🌐 Add Portuguese translation for docs/pt/docs/advanced/dataclasses.md#12475
alejsdev merged 3 commits into
fastapi:masterfrom
leoscarlato:dataclass-translation-pt

Conversation

@leoscarlato

Copy link
Copy Markdown
Contributor

No description provided.

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

Copy link
Copy Markdown
Contributor

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

Copy link
Copy Markdown
Contributor

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

@ceb10n

ceb10n commented Oct 18, 2024

Copy link
Copy Markdown
Contributor

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

off course \o/

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

Just a missing translation in the admonition and some other suggestions 😄

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

Isso funciona da mesma forma que com os modelos Pydantic. E na verdade é alcançado da mesma maneira por baixo dos panos, usando Pydantic.

/// info

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.

it's nice to add translation to info:

/// info | Informação

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

Novamente, você pode combinar `dataclasses` com anotações de tipo padrão.

8. Note que esta *função de operação de caminho* usa `def` regular em vez de `async def`.

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.

função de operação de caminho is right, but the pattern used in the pt translations is função de operação de rota.

It would be nice to keep the pattern 😄

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

Se você precisar de uma atualização sobre quando usar qual, confira a seção _"Com pressa?"_ na documentação sobre [`async` e `await`](../async.md#in-a-hurry){.internal-link target=_blank}.

9. Esta *função de operação de caminho* não está retornando dataclasses (embora pudesse), mas uma lista de dicionários com dados internos.

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.

The same here: função de operação de rota

@leoscarlato

leoscarlato commented Oct 18, 2024

Copy link
Copy Markdown
Contributor Author

Thanks for the feedback, @ceb10n! Will work on these corrections now.

@github-actions

Copy link
Copy Markdown
Contributor

@alejsdev alejsdev merged commit bbcee4d into fastapi:master Oct 22, 2024
@alejsdev

Copy link
Copy Markdown
Member

Thank you! @leoscarlato 🙌 🚀

And thanks to everyone for the reviews 🔍

s-rigaud pushed a commit to s-rigaud/fastapi that referenced this pull request Jan 23, 2025
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.

5 participants