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.

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

Add some msgstr translations #1364

Merged
merged 8 commits into from
Dec 3, 2018
Merged

Add some msgstr translations #1364

merged 8 commits into from
Dec 3, 2018

Conversation

yeyeto2788
Copy link

Help on issue #1094

Copy link
Collaborator

@sabas1080 sabas1080 left a comment

Choose a reason for hiding this comment

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

Gracias @yeyeto2788 he agregado algunas notas por favor revisar y decirme que te parecen

locale/es.po Outdated Show resolved Hide resolved
locale/es.po Outdated Show resolved Hide resolved
locale/es.po Outdated Show resolved Hide resolved
locale/es.po Outdated Show resolved Hide resolved
locale/es.po Outdated Show resolved Hide resolved
@yeyeto2788
Copy link
Author

@sabas1080 muchas gracias por hacer la revision, he modificado lo que me comentaste, de todos modos si ves algo mas no dudes en contactar conmigo. El hecho otro commit con los arreglos.

Saludos!

@C47D
Copy link
Collaborator

C47D commented Nov 30, 2018

Creo que Travis esta en rojo porque en la línea 1900 sobra un espacio al final de la traducción.

@yeyeto2788
Copy link
Author

Hola @C47D, en toeria pensaba que concatenaba las frases. O dices después de las comillas?

locale/es.po Show resolved Hide resolved
@C47D
Copy link
Collaborator

C47D commented Nov 30, 2018

We also need to ask @tannewt if here are not python keywords being translated (see #1098).

sabas1080
sabas1080 previously approved these changes Nov 30, 2018
@sabas1080
Copy link
Collaborator

Resuelva el problema con travis y todo ok 👍

@C47D
Copy link
Collaborator

C47D commented Dec 1, 2018

Sigue fallando un trabajo de Travis, los demás estan verdes, revise el log y aparece este error en todas las traducciones, supongo que es el causante de que no pase el trabajo que falta.

imagen

@tannewt does the error on the image above is causing the last Travis job to fail?

@yeyeto2788
Copy link
Author

@C47D he intentado obtener los últimos cambios a mí repositorio a ver si eso lo soluciona, he visto que hay algunos conmuta que se han hecho arreglando cosas referentes a Bluetooth.
Por casualidad has probado los cambios? Hay alguna manera de compilar el código con el lenguaje y verlo directamente en la placa?
Esperemos que eso lo solucione.

@tannewt I did try getting latest changes done on the main repository (adafruit one) since I saw some fixes regarding BLE. Hope it helps 🙏 .

Regards.

@yeyeto2788
Copy link
Author

yeyeto2788 commented Dec 2, 2018

Wuju!!! 😄 It did pass.

The weird thing is that it is no longer accepted the PR 😢

@C47D
Copy link
Collaborator

C47D commented Dec 2, 2018

@yeyeto2788 Estoy por comprar una tarjeta de Adafruit para poder correr Circuitpython en ella :D, quiero la que trae el chip BLE pero no se como este el tema de importaciones, la última vez que intenté importar una tarjeta con BLE me pidieron mucha info para aduanas.

Aqui @tannewt explica como compilar Circuitpython o puedes obtener el binario una vez que se mergeo tu PR.

@C47D
Copy link
Collaborator

C47D commented Dec 3, 2018

@yeyeto2788 Pregunté en Discord y todo perfecto, me parece que falta la aprovación de @sabas1080 para poder mergear a master!!!

@sabas1080
Copy link
Collaborator

@C47D @yeyeto2788 yo puedo enviar a cada uno una de mis tarjetas que soportan circuitpython unas bast pro mini, usb stick o meow meow si gustan de regalo ;)

@C47D
Copy link
Collaborator

C47D commented Dec 3, 2018

@sabas1080 Gracias por la oferta, estaría perfecto!

@yeyeto2788
Copy link
Author

yeyeto2788 commented Dec 3, 2018

@C47D muchas gracias por esa informacion de como poder correr CircuitPython en otras placas. Actualmente solo tengo linux, asi que lo probare alli de momento. 😛

@sabas1080, muchas gracias. eso serial genial!

@tannewt
Copy link
Member

tannewt commented Dec 3, 2018

I think we're all set. Thanks @yeyeto2788 for the PR and @sabas1080 and @C47D for the review.

@tannewt tannewt merged commit 7a52712 into adafruit:master Dec 3, 2018
@yeyeto2788
Copy link
Author

@tannewt, It was a pleasure! I hope with this we can get more people involve on programming. If any other help is needed do not hesitate on contacting me.

@sabas1080, @C47D thank you very much for all the help.

@tannewt
Copy link
Member

tannewt commented Dec 3, 2018

@yeyeto2788 I hope so too! Thanks again.

@sabas1080
Copy link
Collaborator

@yeyeto2788 @C47D pésenme su dirección a s[arroba]electronicccats[punto]com por favor y diganme que tarjeta quiere esta semana llega la nueva Bast

@yeyeto2788
Copy link
Author

@sabas1080 he intentado enviarte el correo pero me dice que el dominio no pudo ser encontrado al enviar el correo. 😢

@C47D
Copy link
Collaborator

C47D commented Dec 4, 2018

@yeyeto2788 le sobra una c después del arroba, es electroniccats

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants