Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

馃寪 Add Spanish translation for docs/es/docs/how-to/graphql.md #11697

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

camigomezdev
Copy link

No description provided.

@tiangolo
Copy link
Owner

tiangolo commented Jun 7, 2024

馃摑 Docs preview for commit 50224d0 at: https://0a3f26fd.fastapitiangolo.pages.dev

@ajpinedam
Copy link

@tiangolo This PR came from the GraphQL presentation at PyCon Colombia. It's nice to see someone step up and complete this translation, which will help others.

Thank you @camigomezdev

Copy link
Collaborator

@alejsdev alejsdev left a comment

Choose a reason for hiding this comment

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

En general LGTM, faltan un par de tildes y corregir un typo. 馃樃

docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
@alejsdev alejsdev added lang-all Translations lang-es Spanish translations awaiting-review labels Jun 9, 2024
@alejsdev alejsdev changed the title 馃寪 Add Spanish translation for docs/es/docs/how-to/graphql.md 馃寪 Add Spanish translation for docs/es/docs/how-to/graphql.md Jun 9, 2024
@tiangolo
Copy link
Owner

tiangolo commented Jun 9, 2024

馃摑 Docs preview for commit 74aa87a at: https://c58184f8.fastapitiangolo.pages.dev

Copy link
Collaborator

@alejsdev alejsdev left a comment

Choose a reason for hiding this comment

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

Hola! Es un detalle menor y creo que lo pas茅 por alto en el review anterior, pero cuando el "si" es condicional no lleva tilde.

Adem谩s, normalmente en la documentaci贸n se ha dejado la palabra "library" en ingl茅s, ya que la traducci贸n "librer铆a" es incorrecta. No s茅 si se deber铆a ajustar eso en este caso, ya que la palabra se usa varias veces. Thoughts? @tiangolo 馃

docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
docs/es/docs/how-to/graphql.md Outdated Show resolved Hide resolved
@tiangolo
Copy link
Owner

Adem谩s, normalmente en la documentaci贸n se ha dejado la palabra "library" en ingl茅s, ya que la traducci贸n "librer铆a" es incorrecta. No s茅 si se deber铆a ajustar eso en este caso, ya que la palabra se usa varias veces. Thoughts? @tiangolo 馃

Sip, de acuerdo con @alejsdev, tal cual todo 馃挴 Gracias! 馃

@camigomezdev camigomezdev force-pushed the master branch 2 times, most recently from e486da5 to 73c5d12 Compare June 14, 2024 21:22
@camigomezdev
Copy link
Author

Quedo atenta si hay mas comentarios :)

@tiangolo
Copy link
Owner

馃摑 Docs preview for commit 246b6d0 at: https://a1c69998.fastapitiangolo.pages.dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
awaiting-review lang-all Translations lang-es Spanish translations
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants