-
Notifications
You must be signed in to change notification settings - Fork 229
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 new lesson on map warper ticket #212 #1845
Conversation
No acabo de entender cuáles son los errores que indica Travis @rivaquiroga - lo dejo en tus manos, pero si necesitas ayuda o debo encargarme yo de resolverlos, dímelo, no sé exactamente cuál es procedimiento porque hace bastante que no publico una lección. |
Acabo de descubrir una cosa que me inquieta: los autores han usado links para las imágenes en lugar de includes... por ejemplo: En este caso se trata de un gif... pero a en otros casos son imágenes que están guardadas en /images y que no están incluidas con include propiamente. ¿Qué pensáis @rivaquiroga @jenniferisasi ? |
Otra cosa que me inquieta, los autores han incluido una lista mediante un enlace: https://docs.google.com/spreadsheets/d/1Lm_S6E8JfPSHrmcimBNye3g7QcfQ8xe-fUx8n_Nr8iY/edit#gid=0 Se tendría que descargar y guardar en assets no? |
Lo de las imágenes enlazadas y en formato gif me inquieta porque la lección se ve bien como podéis ver aquí: https://github.com/programminghistorian/jekyll/blob/add-new-original-lesson-212/es/lecciones/introduccion-map-warper.md Sin embargo, este procedimiento no es el habitual y sospecho que los errores de Travis están provocados por esto. ¿Qué debemos hacer? Gracias |
@arojascastro, mientras la lección no tenga DOI el build va a fallar. Revisaré si no hay otro problema además de eso |
@arojascastro, en todo caso, no es necesario que como editor te preocupes de abrir el PR. Así que yo me encargo de que todo funcione de aquí en adelante. |
@arojascastro, arreglé los problemas que había (formato en el YAML de autores y forma de referenciar las imágenes que están en |
Gracias @rivaquiroga por arreglar lo de las imágenes, espero que no haya sido mucho trabajo... en cuanto a la lista: creo que es mejor descargar y ponerla en assets. ¿Quieres que les escriba a los autores para que nos den permiso para descargarla (creo que no tenemos credenciales)? |
@arojascastro, el documento de Google está con acceso abierto, por lo que puedes descargarlo directamente como .csv. No es necesario pedirle a los autores que lo hagan. |
Gracias @rivaquiroga ya lo añadí: todo tuyo! |
@arojascastro, está todo listo para hacer merge. Una vez que eso esté listo, tengo que cerrar el proceso del DOI con la biblioteca de Sussex para que se pueda acceder al tutorial desde https://doi.org/10.46430/phes0048. Así que si todo sale rápido, mañana miércoles ya podríamos anunciar la nueva lección. |
I am starting to prepare the publication of lesson reviewed in ticket 212: programminghistorian/ph-submissions#212
Checklist
Closes #ISSUENUMBER
to the description aboveIf you are having difficulty fixing Travis errors, first consult https://github.com/programminghistorian/jekyll/wiki/Making-Technical-Contributions carefully, especially "Common Travis Errors". Then contact the technical team if you need further help.