Refactor - Githubissues

mgaitan commented 7 years ago

Implementacion (incompleta) del refactor de modelos descripto en #28, para soportar la carga de multiples categorias en una mesa, con opciones independientes entre sí.

falta el refactor en el dashboard de resultados y analizar criterios de usabilidad.

almanuel commented 7 years ago

Tuve errores para levantarlo. Me parece que hay que actualizar el data.json

almanuel commented 7 years ago

Estuve mirando el código y probando la aplicación (sin docker). Levantó bien, importa bien los datos. Hay algunas cosas que parecen no funcionar como la transcripción de datos, donde me muestra solo la categoría Senadores, sin input de texto/numero y se rompe al darle Siguiente.

Creo que podríamos arreglar esto y estaríamos listos para mergear. Quizas haya algun otro bug, pero lo importante del refactor es el cambio en el modelo.

mgaitan commented 7 years ago

@manuelpira porfa, pegame el traceback del error que te da,

almanuel commented 7 years ago

Así veo el form de carga: cargar_siguiente

Error al clickear en Siguiente.

Internal Server Error: /cargar/8
Traceback (most recent call last):
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/django/core/handlers/exception.py", line 41, in inner
    response = get_response(request)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/django/core/handlers/base.py", line 187, in _get_response
    response = self.process_exception_by_middleware(e, request)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/django/core/handlers/base.py", line 185, in _get_response
    response = wrapped_callback(request, *callback_args, **callback_kwargs)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/django/views/generic/base.py", line 68, in view
    return self.dispatch(request, *args, **kwargs)
  File "/home/manu/dev/gaitan/escrutinio-social/fiscales/views.py", line 285, in dispatch
    return super().dispatch(request, *args, **kwargs)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/django/contrib/auth/mixins.py", line 56, in dispatch
    return super(LoginRequiredMixin, self).dispatch(request, *args, **kwargs)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/formtools/wizard/views.py", line 244, in dispatch
    response = super(WizardView, self).dispatch(request, *args, **kwargs)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/django/views/generic/base.py", line 88, in dispatch
    return handler(request, *args, **kwargs)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/formtools/wizard/views.py", line 308, in post
    return self.render_next_step(form)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/formtools/wizard/views.py", line 322, in render_next_step
    files=self.storage.get_step_files(next_step),
  File "/home/manu/dev/gaitan/escrutinio-social/fiscales/views.py", line 322, in get_form
    formset = super().get_form(step, data, files)
  File "/home/manu/dev/gaitan/escrutinio-social/escrutinio-gaitan/lib/python3.6/site-packages/formtools/wizard/views.py", line 412, in get_form
    form_class = self.form_list[step]
KeyError: '1'

mgaitan commented 7 years ago

@manuelpira ahi al parecer el tema es que tenes una categoria sin ninguna opcion asociada, porque sino deberia aparecerte ahi en el formset para cargar.

almanuel commented 7 years ago

No se pero ambas categorías tienen todas las opciones seleccionadas. Ahora lo probé en otra maquina, usando docker y sigue igual... :thinking:

mgaitan commented 7 years ago

proba instalar esa version modificada de formtools que pushee al requirements

git+https://github.com/mgaitan/django-formtools#egg=django-formtools

On Mon, Oct 2, 2017 at 5:30 PM, Manuel Alonso notifications@github.com wrote:

No se pero ambas categorías tienen todas las opciones seleccionadas. Ahora lo probé en otra maquina, usando docker y sigue igual... 🤔

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/democraciaconcodigos/escrutinio-social/pull/37#issuecomment-333655628, or mute the thread https://github.com/notifications/unsubscribe-auth/ACPyB6XWmaRHYfUcH-csjY8jqWespENoks5soUfYgaJpZM4PlHvv .

-- mgaitan.github.io textosypretextos.com.ar

almanuel commented 7 years ago

Levanté con el docker, levantó esos nuevos requirements pero sigue sin funcionar :cry:

rulosant commented 7 years ago

Tengo el mismo error. La categoría tiene opciones

mgaitan commented 7 years ago

lo miro a la tarde muchachos

On Wed, Oct 4, 2017 at 10:38 AM, rulosant notifications@github.com wrote:

Tengo el mismo error. La categoría tiene opciones

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/democraciaconcodigos/escrutinio-social/pull/37#issuecomment-334157791, or mute the thread https://github.com/notifications/unsubscribe-auth/ACPyB6lcbl3WIDryic38Pc_BlV1_IF7gks5so4pQgaJpZM4PlHvv .

-- mgaitan.github.io textosypretextos.com.ar

democraciaconcodigos / escrutinio-social

Refactor #37