search

civio / quiencobralaobra

¿Quién cobra la obra?
GNU Affero General Public License v3.0
7 stars 0 forks source link

Definir la id de QM en los mayores grupos constructores para añadir si grafo de QM #79

Closed rauldiazpoblete closed 7 years ago

rauldiazpoblete commented 7 years ago

Mi primera intención ha sido añadir una propiedad qm_id en Bidders para definir desde el admin la id de QM de los mayores grupos. Pero me he dado cuenta de que en realidad no tenemos ninguna entidad Grupo sino varias entidades Empresa que comparten la propiedad group, así que no veo la manera de hacerlo desde el admin. La única alternativa que se me ocurre es añadir un hash de ids de QM a pelo en el controller de Bidders y recoger la id a partir del nombre de cada grupo.

rauldiazpoblete commented 7 years ago

He añadido en https://github.com/civio/quiencobralaobra/commit/22b0f1e7a104324234abdc083726d042850f57d0 un hash en BiddersController que mapea los slugs de los "Reyes del ladrillos" (ACS, Acciona, Isolux, Ferrovial, FCC, Sacyr, Villar Mir, Aldesa, San José y Comsa) con sus ids de QM. En aquellos Grupos con id de QM mostramos su gráfico justo delante de la tabla de contratos (si lo ponemos detrás no lo verá nadie). La solución es un poco chusca pero no se me ocurría nada mejor. Cierro el issue y en caso de que encontremos un mejor método lo volvemos a reabrir.

dcabo commented 7 years ago

Me parece bien para salir del paso. 👍