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: traducción de guide/styleguide y guide/migration-renderer #275

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

docs: traducción de guide/styleguide y guide/migration-renderer #275

wants to merge 8 commits into from

Conversation

wilo-dev
Copy link

@wilo-dev wilo-dev commented Oct 13, 2020 •
edited
Loading

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?

Archivo traducido del ingles a español

¿Cuál es el nuevo comportamiento?

Archivo en inglés traducido al español #218

@CLAassistant
Copy link

CLAassistant commented Oct 13, 2020 •
edited
Loading

CLA assistant check
All committers have signed the CLA.

@AntonioCardenas AntonioCardenas requested a review from a team October 13, 2020 21:30
@Splaktar
Copy link
Member

En el futuro, por favor traduzca solo una guía por PR. Esto incluye tanto la guía de estilo como las traducciones de migración del renderizador.

This was linked to issues Oct 16, 2020
Traducir: guide/migration-renderer.md #208
Open
Traducir: guide/styleguide.md #218
Open
@Splaktar Splaktar changed the title angular:master docs: traducción de guide/styleguide y guide/migration-renderer Oct 16, 2020
@devpato devpato added the Pausa label Oct 20, 2020
devpato
devpato requested changes Oct 20, 2020
View reviewed changes
Copy link
Member

@devpato devpato 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.

Favor de creear 2 PRs. Una para migration-render y otra para styleguide como comenta @Splaktar

@devpato devpato removed the Pausa label Oct 20, 2020
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.

Favor de creear 2 PRs. Una para migration-render y otra para styleguide

@wilo-dev
Copy link
Author

wilo-dev commented Feb 2, 2021

Saludos cordiales, tengo 4 errores alguien m podría ayudar indicando como los puede solucionar, gracias de antemano

@gustavguez
Copy link
Member

Hola @willian593 muchas gracias por tu aporte!

Primero deberías intentar que este PR solo afecte a 1 archivo y su correspondiente en.md. Porque aún sigue apareciendo 4 archivos en el PR:

Screen Shot 2021-02-02 at 9 48 53 PM

Saludos!

@dalejodc
Copy link
Member

dalejodc commented Feb 3, 2021

Claro @willian593 , si necesitas ayuda puedes escribirnos en el chat de la comunidad para poder ayudarte. ¡Con gusto te puedo ayudar!

@wilo-dev
Copy link
Author

wilo-dev commented Feb 3, 2021

saludos @gustavguez, disculpa me podrías ayudar con pautas de como puede resolver el problema, puesto q mencionas que tengo q intentar que este PR solo afecte a 1 archivo, y no tengo idea de como hacerlo

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

@gustavguez gustavguez gustavguez requested changes

@devpato devpato devpato 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.

Traducir: guide/styleguide.md Traducir: guide/migration-renderer.md
6 participants
@wilo-dev @CLAassistant @Splaktar @gustavguez @dalejodc @devpato