Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

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

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

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

Conversation

@AnandaCampelo
Copy link
Contributor

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

@github-actionsgithub-actionsbot added the lang-allTranslations labelSep 18, 2024
@github-actions
Copy link
Contributor

@tiangolotiangolo added lang-ptPortuguese translations awaiting-review labelsSep 18, 2024
Copy link
Contributor

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


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

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

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

Copy link
Contributor

@ceb10nceb10n 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 useo before FastAPI:

"Comoo FastAPI é baseado..."

and:

...GraphQL que tambémé compatível como 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 changeno fordo:

"Dependendodo 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 tona documentação do Strawberry

ceb10n reacted with thumbs up emoji
@github-actions
Copy link
Contributor

Copy link
Contributor

@ceb10nceb10n left a comment

Choose a reason for hiding this comment

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

❤️

@alejsdevalejsdev merged commita681aeb intofastapi:masterOct 4, 2024
@alejsdev
Copy link
Member

Thanks for your contribution!@AnandaCampelo 🤓 🚀

Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

4 more reviewers

@marcelomarkusmarcelomarkusmarcelomarkus left review comments

@ceb10nceb10nceb10n approved these changes

@JoaoGustavoRogelJoaoGustavoRogelJoaoGustavoRogel approved these changes

@nazarepiedadynazarepiedadynazarepiedady approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approved-2lang-allTranslationslang-ptPortuguese translations

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@AnandaCampelo@alejsdev@ceb10n@marcelomarkus@JoaoGustavoRogel@nazarepiedady@tiangolo

Comments


[8]ページ先頭

©2009-2026 Movatter.jp