Skip to content

docs: Empezando con una aplicación Angular básica #301

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

Closed
wants to merge 1 commit into from

Conversation

mferreiras
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?

¿Cuál es el nuevo comportamiento?

Ejemplo: Archivo en inglés traducido al español feat(lang): #117

@gustavguez
Copy link
Member

Hola!

Para solucionar los Tests del PR que están fallando tienes que actualizar el branch con master, para ello tienes 2 opciones:

Opción 1: Merge, pero esto crea un commit extra

git checkout feature-branch
git merge origin/master
git push origin feature-branch

Opción 2: Rebase

git fetch
git rebase origin/master

Una vez realizado esto, seguiremos con el review correspondiente.

Gracias!

@Splaktar Splaktar changed the title docs: index.md Traducido al Español docs: Empezando con una aplicación Angular básica Jan 8, 2021
@Splaktar Splaktar linked an issue Jan 8, 2021 that may be closed by this pull request
Copy link
Member

@gustavguez gustavguez left a comment

Choose a reason for hiding this comment

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

Hola @mferreiras Gracias por tu aporte!

Es necesario el archivo original con la extensión .en.md y debes de actualizar el branch con master como sugerí en el comment pasado así los test dan success.

Saludos!

@ealexisaraujo
Copy link

@Splaktar I would like to work with this issue could you please assign me this issue? Please?

@Splaktar
Copy link
Member

@ealexisaraujo Te envié una invitación para el equipo. Una vez que lo aceptes, podré asignarte #117.

@ealexisaraujo
Copy link

ealexisaraujo commented Jun 14, 2021

@ealexisaraujo Te envié una invitación para el equipo. Una vez que lo aceptes, podré asignarte #117.

Listo ya acepte la invitacion a la organizacion @Splaktar

@mzguido
Copy link

mzguido commented Apr 8, 2022

@Splaktar @gustavguez veo que el issue aun no tiene solución, podrían invitarme al equipo y asignarme el issue?

@Splaktar
Copy link
Member

Voy a cerrar esto ya que nunca se respondió al cambio solicitado de #301 (review).

@Splaktar Splaktar closed this Apr 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Traducir: start/index.md
6 participants