-
Notifications
You must be signed in to change notification settings - Fork 396
Traducido archivo {queue.po} #459
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
Conversation
Gracias @camilooob por la traducción! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No terminé el archivo pero te dejé un par de casos que se repiten en el archivo que te van a ayudar a solucionar los problemas de la build. Gracias! :)
Hola @camilooob! Cristián te hizo unas sugerencias, ¿puedes revisarlas a ver qué te parecen? |
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tienes que pasarle "powrap" al archivo localmente, y va a mover un par de palabras y eso de seguro quita el error de travis.
Complementariamente, si pudieras echarle un vistado a las referencias de "devolver" y usar "retornar" seria guay :)
@camilooob a ver si con las sugerencias mejora el build. Avisanos si te podemos ayudar en algo mas! |
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
Co-authored-by: Cristián Maureira-Fredes <cmaureir@users.noreply.github.com>
uuuuh ya casi casi! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Gracias @camilooob encontré un par de detalles en la traducción, de cosas que de seguro se te pasaron, y temas de comillas :) yo creo que luego de esto, estaremos mucho más cerca para el merge!
@camilooob te podemos ayudar en algo para continuar con el PR? |
༼ つ ◕_◕ ༽つ |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
¡Gracias @camilooob por la traducción y @cmaureir por retomar el archivo!
Dejé algunas sugerencias de forma y otras de palabras que creo se deberían estandarizar en el archivo, espero solucionando esto estemos más cerca del merge :)
Co-authored-by: narvmtz <51009725+narvmtz@users.noreply.github.com>
Gracias @narvmtz por la revisión, avísame si te parece bien para hacer el merge :) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
¡@cmaureir gracias por el trabajo en este archivo! Ya estamos listos para merge 🎉
🎉 |
Closes #428