Skip to content

🌐 Add Spanish translation for docs/tutorial/response-status-code.md#5466

Closed
carlosm27 wants to merge 10 commits intofastapi:masterfrom
carlosm27:master
Closed

🌐 Add Spanish translation for docs/tutorial/response-status-code.md#5466
carlosm27 wants to merge 10 commits intofastapi:masterfrom
carlosm27:master

Conversation

@carlosm27
Copy link

Hi folks, I hope all of you are well.

According to the Spanish Translation feature: #1218
This pull request includes the Spanish translation for tutorial/response-status-code.md

Thanks!

@carlosm27 carlosm27 changed the title Add Spanish traslation for docs/tutorial/response-status-code.md Add Spanish traslation for docs/tutorial/response-status-code.md Oct 6, 2022
@codecov
Copy link

codecov bot commented Oct 6, 2022

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (cf73051) 100.00% compared to head (0c4cbed) 100.00%.
Report is 1086 commits behind head on master.

❗ Current head 0c4cbed differs from pull request most recent head b37bb84. Consider uploading reports for the commit b37bb84 to get more accurate results

Additional details and impacted files
@@            Coverage Diff            @@
##            master     #5466   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          540       540           
  Lines        13969     13946   -23     
=========================================
- Hits         13969     13946   -23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@github-actions
Copy link
Contributor

github-actions bot commented Oct 6, 2022

📝 Docs preview for commit 7acfff1 at: https://633f092fb3ac08020cc9078d--fastapi.netlify.app

@carlosm27
Copy link
Author

I don't know why tests failed. I saw the test's details and they say the test fails in the file test/test_tutorial/test_async_tests/test_main.py. Can someone explain to me why this happened? I think I didn't change other files but the one I was working on.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 020a7c3 at: https://634430357c44910d6f811083--fastapi.netlify.app

@github-actions
Copy link
Contributor

📝 Docs preview for commit 0c4cbed at: https://6357303925f221551d1eb4c7--fastapi.netlify.app

@tiangolo tiangolo changed the title Add Spanish traslation for docs/tutorial/response-status-code.md 🌐 Add Spanish translation for docs/tutorial/response-status-code.md Oct 31, 2022
@github-actions github-actions bot mentioned this pull request Oct 31, 2022
@github-actions
Copy link
Contributor

📝 Docs preview for commit 15e2f6c at: https://639cc8727626050e7d0703ac--fastapi.netlify.app

Copy link

@erudin erudin left a comment

Choose a reason for hiding this comment

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

Looks good! Thanks! Just a few small comments regarding missing accents and some rephrasing for clarity.

@@ -0,0 +1,105 @@
# Response Status Code

De la misma manera que puedes especificar un modelo de respuesta, también puedes declarar codigos de estado HTTP
Copy link

Choose a reason for hiding this comment

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

Suggested change
De la misma manera que puedes especificar un modelo de respuesta, también puedes declarar codigos de estado HTTP
De la misma manera que puedes especificar un modelo de respuesta, también puedes declarar códigos de estado HTTP

# Response Status Code

De la misma manera que puedes especificar un modelo de respuesta, también puedes declarar codigos de estado HTTP
usados para la respuesta con el parametro `status_code` en cualquiera de los *path operations*:
Copy link

Choose a reason for hiding this comment

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

Suggested change
usados para la respuesta con el parametro `status_code` en cualquiera de los *path operations*:
usados para la respuesta con el parámetro `status_code` en cualquiera de los *path operations*:

```

!!! note
Toma en cuenta que `status_code` es un parámetro del método del "decorador" (`get`, `post`, etc)-
Copy link

Choose a reason for hiding this comment

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

Suggested change
Toma en cuenta que `status_code` es un parámetro del método del "decorador" (`get`, `post`, etc)-
Ten en cuenta que `status_code` es un parámetro del método del "decorador" (`get`, `post`, etc)-

Comment on lines +26 to +29
Será:

- Retorna ese código de estado en la respuesta.
- Documentado como en el OpenAPI "schema" (además, en la interfaz de usuario):
Copy link

Choose a reason for hiding this comment

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

I'm trying to think in another way to say this part. Because it's a bit confusing, and the meaning is a bit lost.
What do you think about something like this?:

Esto:

  • Devolverá ese código de estado en la respuesta.
  • Lo documentará como tal en el esquema OpenAPI (y por lo tanto, en las interfaces de usuario):



!!! note
Algunos códigos de respuesta (ve la siguiente sección) indica que la respuesta no tiene body.
Copy link

Choose a reason for hiding this comment

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

Suggested change
Algunos códigos de respuesta (ve la siguiente sección) indica que la respuesta no tiene body.
Algunos códigos de respuesta (ve la siguiente sección) indica que la respuesta no tiene cuerpo.

- `300` en adelante son para "Redirección". Respuestas con estos códigos de estado pueden o no tener body, excepto para `304`, "No Modificado",
el cual no debe tener body.

- `400` en adelante son para respuestas para `Error del Cliente`. Estos son el segundo tipo de codigós que más usarás.
Copy link

Choose a reason for hiding this comment

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

Suggested change
- `400` en adelante son para respuestas para `Error del Cliente`. Estos son el segundo tipo de codigós que más usarás.
- `400` en adelante son para respuestas de `Error del Cliente`. Estos son el segundo tipo de codigós que probablemente más usarás.

- Un ejemplo es `404`, para respuesta de "No encontrado".
- Para errores genéricos desde el cliente, puedes usar `400`.

- `500` en adelante son para errores del servidor. Tú nunca los usas directamente. Cuando algo sale mal en alguna parte del código de tu aplicación,
Copy link

Choose a reason for hiding this comment

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

Suggested change
- `500` en adelante son para errores del servidor. Tú nunca los usas directamente. Cuando algo sale mal en alguna parte del código de tu aplicación,
- `500` en adelante son para errores del servidor. Tú casi nunca los usas directamente. Cuando algo sale mal en alguna parte del código de tu aplicación,

{!../../../docs_src/response_status_code/tutorial002.py!}
```

Son tan convenientes, mantienen el mismo número, pero puedes usar el autocompletado del editor para encontrarlas:
Copy link

Choose a reason for hiding this comment

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

Suggested change
Son tan convenientes, mantienen el mismo número, pero puedes usar el autocompletado del editor para encontrarlas:
Son solo una conveniencia, mantienen el mismo número, pero puedes usar el autocompletado del editor para encontrarlas:



```
**FastAPI** provee el mismo `starlette.status` como `fastapi.status` conveniente para tí, el desarrollador. Pero viene directamente de Starlette.
Copy link

Choose a reason for hiding this comment

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

Suggested change
**FastAPI** provee el mismo `starlette.status` como `fastapi.status` conveniente para tí, el desarrollador. Pero viene directamente de Starlette.
**FastAPI** provee el mismo `starlette.status` como `fastapi.status` solo como una conveniencia para tí, el desarrollador. Pero viene directamente desde Starlette.


# Cambiando el por defecto

Después, en la [Guía Avanzada de Usuario](https://github.com/carlosm27/fastapi/blob/master/docs/en/docs/advanced/response-change-status-code.md), verás cómo retornar diferentes códigos de estado que los de por defecto declaras aquí.
Copy link

Choose a reason for hiding this comment

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

Suggested change
Después, en la [Guía Avanzada de Usuario](https://github.com/carlosm27/fastapi/blob/master/docs/en/docs/advanced/response-change-status-code.md), verás cómo retornar diferentes códigos de estado que los de por defecto declaras aquí.
Después, en la [Guía Avanzada de Usuario](https://github.com/carlosm27/fastapi/blob/master/docs/en/docs/advanced/response-change-status-code.md), verás cómo retornar un código de estado diferente al predeterminado que declaras aquí.

Choose a reason for hiding this comment

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

Isn't there any problem with the link to the forked repo?

Algunos códigos de respuesta (ve la siguiente sección) indica que la respuesta no tiene body.

```
FastAPI sabe esto, y producirá documentatión OpenAPI diciendo que no hay body en la respuesta.

Choose a reason for hiding this comment

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

Suggested change
FastAPI sabe esto, y producirá documentatión OpenAPI diciendo que no hay body en la respuesta.
FastAPI sabe esto, y producirá documentación OpenAPI diciendo que no hay cuerpo en la respuesta.

El parámetro `status_code` recibe un número con el código del estado HTTP.

!!! info
`status_code` puede alternamente recibir un `IntEnum`, como <a href="https://docs.python.org/3/library/http.html#http.HTTPStatus" class="external-link" target="_blank">`http.HTTPStatus`</a> de Python.

Choose a reason for hiding this comment

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

Suggested change
`status_code` puede alternamente recibir un `IntEnum`, como <a href="https://docs.python.org/3/library/http.html#http.HTTPStatus" class="external-link" target="_blank">`http.HTTPStatus`</a> de Python.
`status_code` puede alternativamente recibir un `IntEnum`, como <a href="https://docs.python.org/3/library/http.html#http.HTTPStatus" class="external-link" target="_blank">`http.HTTPStatus`</a> de Python.

En resumen:

- `100` en adelante son para "Información". Rara vez los usas directamente. Las respuestas con estos códigos de estado no tienen body.
- `200` en adelante son para respuestas "Exitosas". Esta son las que se usan con mayor frecuencia.

Choose a reason for hiding this comment

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

Here we are talking about "códigos de estado":

Suggested change
- `200` en adelante son para respuestas "Exitosas". Esta son las que se usan con mayor frecuencia.
- `200` en adelante son para respuestas "Exitosas". Estos son las que se usan con mayor frecuencia.


!!! tip
Para saber más acerca de estos códigos de estado y cuál código se usa para cada caso, revisa la
<a href="https://developer.mozilla.org/en-US/docs/Web/HTTP/Status" class="external-link" target="_blank"><abbr title="Mozilla Developer Network"> documentación acerca de los códigos de estados MDN</abbr> .

Choose a reason for hiding this comment

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

It says "códigos de estado" in the previous line:

Suggested change
<a href="https://developer.mozilla.org/en-US/docs/Web/HTTP/Status" class="external-link" target="_blank"><abbr title="Mozilla Developer Network"> documentación acerca de los códigos de estados MDN</abbr> .
<a href="https://developer.mozilla.org/en-US/docs/Web/HTTP/Status" class="external-link" target="_blank"><abbr title="Mozilla Developer Network"> documentación acerca de los códigos de estado MDN</abbr> .


# Cambiando el por defecto

Después, en la [Guía Avanzada de Usuario](https://github.com/carlosm27/fastapi/blob/master/docs/en/docs/advanced/response-change-status-code.md), verás cómo retornar diferentes códigos de estado que los de por defecto declaras aquí.

Choose a reason for hiding this comment

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

Isn't there any problem with the link to the forked repo?

Copy link

@chesstrian chesstrian left a comment

Choose a reason for hiding this comment

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

The entry for the new translated page is missing from docs/es/mkdocs.yml.

@github-actions
Copy link
Contributor

📝 Docs preview for commit 9e4e2e2 at: https://63fd69dc9c8fe5334fd88386--fastapi.netlify.app

@chesstrian
Copy link

@carlosm27 is it intended not to make changes over docs/es/mkdocs.yml?

@carlosm27
Copy link
Author

Hey @chesstrian. Honestly, I don't know. I think we make changes when a topic is added.

@chesstrian
Copy link

@carlosm27 I mean this: https://fastapi.tiangolo.com/contributing/#tips-and-guidelines
image

This is an example: https://github.com/tiangolo/fastapi/pull/9214/files#diff-03213f8a7ff7438ed5bbe5e7cfd9b2b9d6b959535380af941b07565ec20ba90eR69

Basically any PR that translates a new page needs to update docs/es/mkdocs.yml with the new entry.

@carlosm27
Copy link
Author

Hey @chesstrian Thank you for sharing this. I didn't know about updating that file.

@github-actions
Copy link
Contributor

📝 Docs preview for commit b37bb84 at: https://640e610576710f62ad103237--fastapi.netlify.app

@tiangolo
Copy link
Member

This was covered in another PR, so I'll close this one now, thanks for the effort! 🍰 ☕

@tiangolo tiangolo closed this Dec 30, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants