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.

Sign up for GitHub

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

Jump to bottom

docs: soluciona errores en aio/content/navigation.json #358

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

docs: soluciona errores en aio/content/navigation.json #358

wants to merge 2 commits into from

Conversation

dalejodc
Copy link
Member

Lista de Verificación del PR

Comprueba si tu PR cumple los siguientes requisitos:

  • El mensaje de commit esta conforme con nuestras reglas
  • Probe los cambios que agregué (arreglo de bugs / funcionalidades)
  • Revisé previamente las traducciones o cambios de contenido
  • Consulté el diccionario de términos en español
  • He creado dos archivos con la extensión correspondiente(.en.md para el archivo en inglés y .md para el Archivo en español)
  • He enlazado el commit con el issue correspondiente

Tipo de PR

¿Qué tipo de cambio introduce este PR?

  • Bugfix
  • Funcionalidad
  • Actualización de el estilo del código (formato, variables locales)
  • Refactorización (no cambios en la funcionalidad, no cambios en el api)
  • Cambios relacionados al build
  • Cambios relacionados al CI (Integración continua)
  • Cambios en el contenido de la documentación
  • Cambios en la aplicación / infraestructura de angular.io
  • Otro... Por favor describe la:

¿Cuál es el comportamiento actual?

  • Algunas oraciones están escritas en forma de expresión usted.
  • Faltan algunas traducciones.
  • Faltan algunas tildes.
  • Algunos typos.

¿Cuál es el nuevo comportamiento?

  • Oraciones escritas en forma de expresión "tu".
  • Se agregaron traducciones faltantes.
  • Se corrigieron las palabras a las que les faltaban tildes.
  • Se corrigieron algunos typos.

Fixes: #357

@dalejodc dalejodc requested a review from a team April 30, 2021 04:40
ealexisaraujo
ealexisaraujo requested changes Jul 22, 2021
View reviewed changes
Copy link

@ealexisaraujo ealexisaraujo left a comment •
edited
Loading

Choose a reason for hiding this comment

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

Te deje un cambio para que lo puedas revisar @dalejodc

aio/content/navigation.json Outdated Show resolved Hide resolved
@ealexisaraujo
Copy link

@dalejodc Tambien en el template del PR faltaria marcar el check por que si estan los dos archivos no crees

  • He creado dos archivos con la extensión correspondiente(.en.md para el archivo en inglés y .md para el Archivo en español)

@dalejodc
Copy link
Member Author

Buena observación @ealexisaraujo , gracias. He realizado el cambio sugerido.

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

@ealexisaraujo ealexisaraujo Awaiting requested review from ealexisaraujo

Requested changes must be addressed to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

solucionar: aio/content/navigation.json
2 participants
@dalejodc @ealexisaraujo