8000 🌐 Add Portuguese translation for `docs/pt/docs/how-to/graphql.md` by AnandaCampelo · Pull Request #12215 · fastapi/fastapi · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@AnandaCampelo
Copy link
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
Contributor

@tiangolo tiangolo added lang-pt Portuguese translations awaiting-review labels Sep 18, 2024
Copy link
Contributor
@JoaoGustavoRogel JoaoGustavoRogel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great text!!


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

/// tip
Copy link
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"
8000


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
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"

Copy link
Contributor
@nazarepiedady nazarepiedady left a comment

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.

Copy link
Contributor
@JoaoGustavoRogel JoaoGustavoRogel left a comment

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
Contributor

ceb10n
ceb10n approved these change 8000 s Sep 18, 2024
Copy link
Contributor
@ceb10n ceb10n left a comment

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 🚀


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
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 😃

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


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
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
Contributor

```

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

Copy link
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
Contributor

Copy link
Contributor
@ceb10n ceb10n left a comment

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
Copy link
Member
alejsdev commented Oct 4, 2024

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

0