-
Notifications
You must be signed in to change notification settings - Fork 0
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
Agrupaciones de incidencias [batu] #5
Comments
alvaro, entiendo que con lo que has hecho tu valdria tambien para el listado de incidencias no?? Si es asi, lo metes ahi tambien?? :) |
Lo que yo hice era agrupar por estación. Se podría ampliar el helper para que saque solo la última incidencia por estación y línea. Y si hay más, ¿que saque el resto ocultas y un enlace que las muestre por js? |
claro, en la home seria agrupar las incidencias por estación y linea en un espacio de timepo: si se rompe la linea3 en moncloa, y lo twittean 4 personas, que no aparezca la misma incidencia (por misma incidencia entiendo que es linea-estacion) 4 veces repetida en la home, sino solo una y un link para desplegar el resto de incidencias ( las otras tres) |
Estoy mirando lo que hice para ampliar por línea y no me está gustando. Igual es plan de cambiar un poco el modelo para que una estación sea única y tenga varias líneas, y un incidente en lugar de pertenecer solamente a una estación que pertenezca a una estación y una línea. ¿O lo de repetir estaciones se hizo intencionadamente por alguna razón que desconozco? |
Bueno me he puesto manos a la obra cambiando el modelo, esto si lo he metido en una rama "line_station". Si queréis revisar los cambios a ver. Los tests que hay pasan y echando un vistazo en desarrollo parece todo ok. Esto implica que los datos que hay ahora o no sirven o hay que migrarlos. Lo único que se borra es el campo line_id de station que habría que copiarlo al nuevo line_id en incident. |
He pushead esto ya. He agrupado las incidencias por línea y mostrado solo la primera. He metido un enlace "Ver más" que hace toggle de el resto de incidencias, añadiendo esto al application.js:
Luego he sacado a un partial las incidencias de los globos por cada línea, está en incidents/station_marker_line_incidents.html.erb (si así de cortito) Faltaría ver si el html está bien (lo he modificado a mi antojo) y el css para alinear las incidencias. Hay un par de bugs:
Bueno, y creo que ya va bien... |
No description provided.
The text was updated successfully, but these errors were encountered: