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

docs(docs-infra): traducción de guide/router.md #375

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jcmexdev
Copy link

@jcmexdev jcmexdev commented May 8, 2021

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?

El archivo router.md estaba solo en ingles

¿Cuál es el nuevo comportamiento?

Renombre el archivo a router.en.md y agrege la traducción al archivo router.md

@dalejodc dalejodc requested a review from a team May 10, 2021 16:33
Copy link
Member

@Splaktar Splaktar left a comment

Choose a reason for hiding this comment

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

El mensaje de confirmación debe tener "Fix #154" en lugar de "Fix: #154" para que funcione la vinculación automática y el cierre del problema asociado.

@jcmexdev
Copy link
Author

Lista ya realice el cambio, quedo atento a los comentarios

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants