La neurona no funciona en la ficha de una iniciativa

politicalwatch / tipi-uikit

UI Kit of our parliamentary monitoring platform

http://tipiui.politicalwatch.es

0 stars 0 forks source link

La neurona no funciona en la ficha de una iniciativa #2

Closed pr3ssh closed 4 years ago

pr3ssh commented 4 years ago

Revisarlo en el entorno de DEV

dacmail commented 4 years ago

@pr3ssh una pregunta, aquí no faltan ODS para los colores https://github.com/politicalwatch/tipi-frontend/blob/master/src/components/neuron.vue#L41

pr3ssh commented 4 years ago

@dacmail Sí, sí que faltan. Es una cuestión que quería plantearte. Ahora mismo los ODS tienen imágenes y colores relacionados. Las imágenes están en la BD (ver API y los colores están en el código.

¿Qué propones como mejor solución? O lo metemos todo en la BD o lo metemos en un fichero de configuración tanto en parlamento2030.es como en tipiciudadano.es

dacmail commented 4 years ago

Mmmmm, pues en principio da igual que estén en código o en la api, depende más de qué enfoque le queráis dar, si queréis que cada app pueda decidir colores e iconos, entonces lo dejamos en la config y hacemos que la api no envíe nada relacionado con esto (ni imagen ni color), si preferís que estos atributos sean definidos a través de la api y así ser comunes a todas las apps que se hagan usando este origen de datos, entonces mejor enviar tmb el color en la respuesta de la api, no?

En cualquier caso informame del cambio por que esto afecta a neuron y no se si a algún sitio más

pr3ssh commented 4 years ago

Afecta creo que a neurona, two-circles y a la ficha de topics y topic.

Creo que vamos a tirar por guardarlo en la APP porque la referencia a ICON está en la App y no en la API y queda raro que la API devuelva un nombre de un fichero que ella no tiene. Metemos un fichero para parlamento2030.es y tipiciudadano.es con las temáticas y cada icon y color y uno por defecto por si hay algún problema de nombre de temática que no haga match con los que se encuentran en el fichero.

Te encargas de todo eso? Se puede renombrar los ficheros de imágenes para que sea ods1.png en vez del nombre largo como viene ahora y los colores están en https://www.un.org/sustainabledevelopment/es/sustainable-development-goals/

dacmail commented 4 years ago

Perfe, yo me encargo de parametrizar los iconos y colores de temáticas en la app.

Para todo esto sigo sumando horas y te mantengo informado, ok?

pr3ssh commented 4 years ago

Ok!

dacmail commented 4 years ago

El componente TipiNeuron ya recibe como parámetro los estilos de cada topic para ser representados. Cuando me pases todos los subtopics para la config actualizo la app de parlamento2030 y lo aplico allá donde se usen

pr3ssh commented 4 years ago

@dacmail Veo que no termina de funcionar la neurona. Chequealo aquí.

dacmail commented 4 years ago

Pablo, he metido un console.log en tipi-uikit por que no tengo otra forma de hacer pruebas, yo en local, en esa misma iniciativa, lo veo todo bien: https://www.dropbox.com/s/8g0gohme6x9prpv/Captura%20de%20pantalla%202020-01-16%2008.43.15.png?dl=0

Cuando subas los cambios me dices e intento ver más detalles con el debug.

pr3ssh commented 4 years ago

Subido!

dacmail commented 4 years ago

Pablo, sorry, pero esto puede ser el coñazo total al no poder depurar yo en local. Acabo de subir otra versión con más console.logs por que no doy con la tecla, a ti no se te ocurre por que en local sí funciona y en dev no, verdad? Por que solo pasa con esa iniciativa

dacmail commented 4 years ago

Coño, ahora funciona? https://dev.parlamento2030.es/initiatives/b36b8e050e3a507cec8dc880b15b4281fc4538c0