search

AgoraUS-G1-1617 / Frontend

Visualización y gestión de resultados (frontend)
Other
0 stars 1 forks source link

Se ha roto la integración con la API simulada #11

Closed anddonram closed 7 years ago

anddonram commented 7 years ago

Compañeros de EGC... Nuestra API de recuento simulada...sniff... ha muerto. Pero solo en parte. Resulta que estaba refactorizando el código, para lograr al menos conseguir un botón que enlazara a la API de Recuento auténtica, y cuando al fin lo consigo, pues supe que había cambiado varias funciones y los html donde se muestran la información. Por lo tanto, nuestra API, que tenía un esquema diferente, hay que ajustarla al nuevo. Pero, SOLO podemos tocar el router.js de api-recuento, NO el controlador NI los HTML, porque si no, nos quedaríamos en las mismas y nos cargaríamos el que integra con la API externa.

Con las modernas técnicas de resurrección de software enfocadas a API (es decir, tirarse el finde mirando cómo se hacen las queries con MongoDB), es el momento de devolver a la vida a nuestra querida API. ¿Voluntarios?

anddonram commented 7 years ago

He decidido que, si la integración con recuento está hecha, esto no es para tanto, así que le bajo la prioridad.

agubelu commented 7 years ago

Sería conveniente que echárais un vistazo a la rama development de recuento porque han cambiado cosas.

anddonram commented 7 years ago

Dado que veo difícil que esto esté arreglado, que no es prioritario, y que ya podemos conectarnos con la API de recuento externa (incluso en local), podemos decidir si dejamos esto para el próximo hito o para que lo resuelvan el año que viene. ¿Comentarios?

andjimrio commented 7 years ago

Y si la eliminamos?? No sé, pero creo que ahora no aporta nada. Si los del año que viene quieren hacer pruebas sin usar a Recuento, que hagan un json a pelo en un fichero y lo lean. ¿No os parece? Yo lo eliminaría sin más.

josgavrui1 commented 7 years ago

Yo voto por no eliminarla y dejarla tal y como está, si quieren arreglarla ya tienen parte hecha y si no pues que no la usen, arreglarla toma demasiado tiempo y no es algo prioritario en mi opinión

anddonram commented 7 years ago

Yo también abogo por no eliminarla, los del año que viene dudo que tengan la integración con recuento desde el principio (tendrán que desplegarlo y tal), así que les vendrá bien algo que funcione como stub o mock o lo que sea eso.

anddonram commented 7 years ago

dicho eso voy a ponerme con ello, que me aburro algo (y tengo que sumar horas jeje)

anddonram commented 7 years ago

pues ya esta algo arreglado, respecto de cómo estaba antes