Skip to content

🌐 Add Portuguese translation for docs/pt/docs/how-to/graphql.md#12215

Merged
alejsdev merged 7 commits into
fastapi:masterfrom
AnandaCampelo:pt-graphql
Oct 4, 2024
Merged

🌐 Add Portuguese translation for docs/pt/docs/how-to/graphql.md#12215
alejsdev merged 7 commits into
fastapi:masterfrom
AnandaCampelo:pt-graphql

Conversation

@AnandaCampelo

Copy link
Copy Markdown
Contributor

🌐 Add Portuguese translation for docs/pt/docs/how-to/graphql.md

@github-actions github-actions Bot added the lang-all Translations label Sep 18, 2024
@github-actions

Copy link
Copy Markdown
Contributor

@tiangolo tiangolo added lang-pt Portuguese translations awaiting-review labels Sep 18, 2024

@JoaoGustavoRogel JoaoGustavoRogel 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 text!!

Comment thread docs/pt/docs/how-to/graphql.md Outdated

Você pode combinar *operações de caminho* normais do FastAPI com GraphQL na mesma aplicação.

/// tip

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 add translation for this:

/// tip | "Dica"

Comment thread docs/pt/docs/how-to/graphql.md Outdated

Ela foi descontinuada do Starlette, mas se você tem código que a utilizava, você pode facilmente **migrar** para <a href="https://github.com/ciscorn/starlette-graphene3" class="external-link" target="_blank">starlette-graphene3</a>, que cobre o mesmo caso de uso e tem uma **interface quase idêntica**.

/// tip

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 add translation for this:

/// tip | "Dica"

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

Despite the parts that need a little revision, the rest is good enough to be merged after correction is applied.

@JoaoGustavoRogel JoaoGustavoRogel 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 job!

@github-actions

Copy link
Copy Markdown
Contributor

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

Thank You Ananda 💯

I've made just some few considerations. None of them are blocking, and you can change if you feel it's right.

Thank you 🚀

Comment thread docs/pt/docs/how-to/graphql.md Outdated

Como **FastAPI** é baseado no padrão **ASGI**, é muito fácil integrar qualquer biblioteca **GraphQL** também compatível com ASGI.

Você pode combinar *operações de caminho* normais do FastAPI com GraphQL na mesma aplicação.

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.

although "operações de caminho" is also used, the most common translation in the docs is "operações de rota". I think it's nice to keep de standard 😃

Comment thread docs/pt/docs/how-to/graphql.md Outdated
@@ -0,0 +1,62 @@
# GraphQL

Como **FastAPI** é baseado no padrão **ASGI**, é muito fácil integrar qualquer biblioteca **GraphQL** também compatível com ASGI.

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.

I would use o before FastAPI:

"Como o FastAPI é baseado..."

and:

...GraphQL que também é compatível com o ASGI

Comment thread docs/pt/docs/how-to/graphql.md Outdated

Se você precisar ou quiser trabalhar com **GraphQL**, <a href="https://strawberry.rocks/" class="external-link" target="_blank">**Strawberry**</a> é a biblioteca **recomendada** pois tem o design mais próximo ao design do **FastAPI**, ela é toda baseada em **type annotations**.

Dependendo no seu caso de uso, você pode preferir usar uma biblioteca diferente, mas se você me perguntasse, eu provavelmente sugeriria que você experimentasse o **Strawberry**.

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.

I would change no for do:

"Dependendo do seu caso de uso..."

@github-actions

Copy link
Copy Markdown
Contributor

```

Você pode aprender mais sobre Strawberry em <a href="https://strawberry.rocks/" class="external-link" target="_blank">Strawberry documentation</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.

Suggestion. ✨
em Strawberry documentation to na documentação do Strawberry

@github-actions

Copy link
Copy Markdown
Contributor

@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 merged commit a681aeb into fastapi:master Oct 4, 2024
@alejsdev

alejsdev commented Oct 4, 2024

Copy link
Copy Markdown
Member

Thanks for your contribution! @AnandaCampelo 🤓 🚀

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.

7 participants