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

Traducida event-binding.md #269

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

Traducida event-binding.md #269

wants to merge 1 commit into from

Conversation

altope98
Copy link

@altope98 altope98 commented Oct 5, 2020 •
edited by AntonioCardenas
Loading

…vent-bindind.en.md

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)
  • La PR fue enlazada 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?

#127

¿Cuál es el nuevo comportamiento?

@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.

Alvaro Torrente seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

@gustavguez
Copy link
Member

@altope98 debes firmar el CLA, para poder comenzar con tu PR. Saludos y gracias por colaborar!

@AntonioCardenas AntonioCardenas requested a review from a team October 6, 2020 01:50
@altope98
Copy link
Author

altope98 commented Oct 6, 2020

La he firmado como 4 veces ya pero no aparece como que la haya firmado

gustavguez
gustavguez requested changes Jan 30, 2021
View reviewed changes
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.

Podrías probar hacer un revert del commit y ver si esta configurado correctamente el user de github en tu pc local? porque parece como que al momento de hacer el commit no estaba configurado y tomó mal el nombre de usuario.

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

@gustavguez gustavguez gustavguez requested changes

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.
3 participants
@altope98 @CLAassistant @gustavguez