Skip to content

docs: translate animations.md#329

Open
daniloprieto wants to merge 39 commits intoangular-hispano:masterfrom
daniloprieto:docs/issue-16
Open

docs: translate animations.md#329
daniloprieto wants to merge 39 commits intoangular-hispano:masterfrom
daniloprieto:docs/issue-16

Conversation

@daniloprieto
Copy link
Copy Markdown

@daniloprieto daniloprieto commented Apr 16, 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?
translate aio/content/guide/animations.md to spanish

  • 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
  • [ x] Otro... Por favor describe la: translate to Spanish

¿Cuál es el comportamiento actual?

¿Cuál es el nuevo comportamiento?

aio/content/guide/animations.md

@CLAassistant
Copy link
Copy Markdown

CLAassistant commented Apr 16, 2021

CLA assistant check
All committers have signed the CLA.

@dalejodc dalejodc self-requested a review April 19, 2021 22:39
Copy link
Copy Markdown
Member

@dalejodc dalejodc left a comment

Choose a reason for hiding this comment

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

¡Muchas gracias por tu aporte @daniloprieto! 🚀

He hecho observaciones con la forma de expresión. Todo debe de estar expresado en forma de "tu" y no "usted". Puedes marcar la opción de "Commit suggestion" para que sea más facil para ti hacer los cambios. También hice un par de sugerencias con algunas traducciones y algunas tildes.

Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
daniloprieto and others added 24 commits April 22, 2021 15:36
Se cambia el sujeto de 'su' a 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Se cambia 'le dan' por 'brindan a los usuarios'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Se cambia el sujeto por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Se cambias por ´tu´

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Se agrega 'de'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Cambio de sujeto tácito

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Cambio de sujeto por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Cambio de sujeto por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Cambio de sujeto por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
agreagada tilde en 'múltiples' y cambio de sujeto en 'use' por 'usa'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
daniloprieto and others added 6 commits April 22, 2021 15:48
string por cadena de texto

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
corrección ortográfica requerida

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
cambio de 'usted' por 'tu'

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
string por cadena de texto

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
String por cadena de texto

Co-authored-by: Daniel Díaz <36966980+dalejodc@users.noreply.github.com>
@daniloprieto
Copy link
Copy Markdown
Author

Agregadas correciones sugeridas por @dalejodc. Se cambia el sujeto 'usted' por 'tu' ; y se cambia la denominación 'string' por 'cadena de texto'.

Copy link
Copy Markdown
Author

@daniloprieto daniloprieto left a comment

Choose a reason for hiding this comment

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

Accept Changes

@daniloprieto daniloprieto requested a review from dalejodc April 22, 2021 19:07
Copy link
Copy Markdown
Member

@dalejodc dalejodc left a comment

Choose a reason for hiding this comment

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

¡Excelente! @daniloprieto

Me parece bien.

@daniloprieto
Copy link
Copy Markdown
Author

daniloprieto commented Apr 22, 2021 via email

@gustavguez gustavguez added the Listo y Revisado listo para hacer merge label Apr 24, 2021
@gustavguez gustavguez assigned gustavguez and Splaktar and unassigned gustavguez Apr 24, 2021
@Splaktar
Copy link
Copy Markdown
Member

Splaktar commented Apr 27, 2021

Hay un issue para estos cambios?

@Splaktar Splaktar linked an issue Apr 27, 2021 that may be closed by this pull request
Copy link
Copy Markdown
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.

@dalejodc por lo que puedo decir, ¿ninguna de sus sugerencias es visible en la revisión final? https://github.com/angular-hispano/angular/pull/329/files

Comment thread aio/content/guide/animations.md Outdated
Comment thread aio/content/guide/animations.md Outdated
@Splaktar
Copy link
Copy Markdown
Member

OK, miré con más atención y parece que algunas, pero no todas las sugerencias fueron aceptadas. No estoy seguro de por qué, ya que no hubo ningún comentario con una razón.

@Splaktar Splaktar requested a review from dalejodc April 27, 2021 13:23
@dalejodc dalejodc closed this Apr 28, 2021
@dalejodc dalejodc reopened this Apr 28, 2021
Copy link
Copy Markdown
Member

@dalejodc dalejodc left a comment

Choose a reason for hiding this comment

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

Cierto @Splaktar . No todas todos los cambios habían sido aplicados a la versión final. Asumo que fue por equivocación, por lo que he hecho commit de todas las sugerencias que faltaban.

A mi me parece bien.

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

Labels

Listo y Revisado listo para hacer merge

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Traducir: animations.md

5 participants