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’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🌐 Add Spanish translation for docs/es/docs/fastapi-cli.md #11703

Closed

Conversation

camigomezdev
Copy link
Contributor

No description provided.

@tiangolo
Copy link
Member

tiangolo commented Jun 9, 2024

📝 Docs preview for commit 662a30f at: https://ad737aed.fastapitiangolo.pages.dev

@alejsdev alejsdev added lang-all Translations lang-es Spanish translations awaiting-review labels Jun 11, 2024
Copy link
Contributor

@JavierSanchezCastro JavierSanchezCastro left a comment

Choose a reason for hiding this comment

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

Creo que podría sonar mejor

@tiangolo tiangolo requested a review from alejsdev June 12, 2024 23:33
docs/es/docs/fastapi-cli.md Outdated Show resolved Hide resolved
docs/es/docs/fastapi-cli.md Outdated Show resolved Hide resolved
docs/es/docs/fastapi-cli.md Outdated Show resolved Hide resolved
docs/es/docs/fastapi-cli.md Outdated Show resolved Hide resolved
docs/es/docs/fastapi-cli.md Outdated Show resolved Hide resolved
docs/es/docs/fastapi-cli.md Outdated Show resolved Hide resolved
@tiangolo
Copy link
Member

📝 Docs preview for commit f1b644b at: https://59af3d38.fastapitiangolo.pages.dev

@camigomezdev
Copy link
Contributor Author

@alejsdev todos los comentarios fueron resueltos, me cuentas cualquier modificación adicional

Copy link
Contributor

github-actions bot commented Aug 2, 2024

📝 Docs preview for commit 80cbd79 at: https://a7e47766.fastapitiangolo.pages.dev

@svlandeg
Copy link
Member

@alejsdev : are the required changes made? If so - can we move this one to Tiangolo's column for a final review to get it merged?

Copy link
Contributor

Copy link
Member

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

Hi @camigomezdev!

The original FastAPI CLI documentation has been modified recently (fastapi dev and fastapi run). Could you please adjust them so we can check them again and merge them? Thanks!

Once the changes are made we can proceed @svlandeg 🤓


En la mayoría de los casos, tendrías (y deberías tener) un "proxy de terminación" que maneje HTTPS por ti, esto dependerá de cómo implementes tu aplicación; tu proveedor podría hacerlo por ti o podrías necesitar configurarlo tú mismo.

!!! tip
Copy link
Member

@alejsdev alejsdev Aug 19, 2024

Choose a reason for hiding this comment

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

/// tip | Consejo

Este formato cambió recientemente.

Copy link
Contributor

As this PR has been waiting for the original user for a while but seems to be inactive, it's now going to be closed. But if there's anyone interested, feel free to create a new PR.

@github-actions github-actions bot closed this Sep 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved-1 lang-all Translations lang-es Spanish translations waiting
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants