Skip to content
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

Crear y editar perfiles individuales #173

Open
voodoorai2000 opened this issue Mar 28, 2018 · 15 comments
Open

Crear y editar perfiles individuales #173

voodoorai2000 opened this issue Mar 28, 2018 · 15 comments
Assignees

Comments

@voodoorai2000
Copy link
Contributor

Qué

Los administradores deberían poder crear y editar perfiles individuales, incluidos los datos de sus declaraciones

Cómo

  • Añadiendo una sección de administración donde realizar estas operaciones
  • Mantener el link de editar, que ven los administradores, en la vista de cara a usuarios no-admins

Mockups

front_editar_perfil

admin_index

admin_edit

Notas

Los datos de las declaraciones se guardan en db en formato json

Es una posibilidad migrar a una estructura de tablas y columnas, pero esto sería bastante más costoso y seguramente en el futuro, queremos permitir configurar desde admin estructuras flexibles de declaraciones para cada año.

Por lo que la opción de mantener el formato json para crear y editar parece la más razonable

@voodoorai2000 voodoorai2000 changed the title Creare y editar perfiles individuales Crear y editar perfiles individuales Mar 28, 2018
@raul-fuentes
Copy link
Contributor

Hola, actualmente estan funcionanado estos enlaces, solo visibles si eres adminitrador

screenshot from 2018-03-28 11-56-14

¿ los dejo asi o modifico su aspecto para que coincida con el mockup de @decabeza ?

@voodoorai2000
Copy link
Contributor Author

+1 a mantener el botón de editar y mover los demás a la zona admin

@raul-fuentes
Copy link
Contributor

raul-fuentes commented Mar 28, 2018

por cierto editar y crear una persona no piden permisos especiales, lo logico es que un perfil de gestion lo haga, ¿hay algo pensado al respecto (cancancan)?. De momento solo lo voy a poner para admin

@voodoorai2000
Copy link
Contributor Author

De momento hay poca gente gestionado esta aplicación. Sólo admins 👍

@raul-fuentes
Copy link
Contributor

@decabeza este es el aspecto actual del formulario de edicion/creacion de personas

screenshot from 2018-03-28 16-11-43

es un poco enorme para encajarlo en el mock, seria posible ponerlo como pestañas?

@decabeza
Copy link
Contributor

@raul-fuentes sin problema el separarlos en pestañas, pero entonces creo que lo adecuedo sería que cada pestaña tuviese un botón de "Guardar" 😉

@raul-fuentes
Copy link
Contributor

Hola!

Por fin tengo algo que funciona, el hecho de que haya datos serializados esta siendo un dolor :(
¿seria posible convertir los datos serializados de activities_declarations a modelos? Quiza con una herencia de tabla simple para no complicarlo mas de lo necesario.

de todas formas os muestro un par de imagenes de como va el proceso, aun falta el I18n y unas cuantas cosas pero si no voy por el buen camino mejor arreglarlo antes que despues, @decabeza espero tu opinion :)

screenshot from 2018-04-04 16-28-51

screenshot from 2018-04-04 16-29-11

Un saludo

@decabeza
Copy link
Contributor

decabeza commented Apr 4, 2018

Hola @raul-fuentes, ¡tiene muy buena pinta! 😉

Quizás cuando estén las pestañas con las traducciones queden un poco grandes, pero no te preocupes cuando esté terminado le ajusto un poco el CSS para que queden más pequeñas 😌

@voodoorai2000
Copy link
Contributor Author

Hola @raul-fuentes,

Dos datos más que nos han pedido que son importantes poder modificar:

  • Fecha de actualización del perfil (de momento podemos usar el timestamp estándar de rails updated_at)
  • Foto del perfil (la interfaz no hace falta que sea muy elaborada 😌)

@raul-fuentes
Copy link
Contributor

Hola

@decabeza necesito orientacion :). Necesito un datepicker para los campos de fecha, un previsualizador para mostrar la imagen seleccionada para la foto de perfil y un metodo para pasar las validaciones por javascript en el momento de aceptar el formulario.

¿Como lo hago? ¿me traigo codigo desde consul o busco lo que me parezca mas correcto?

Un saludo

@decabeza
Copy link
Contributor

Hola @raul-fuentes,

Para el datepicker puedes usar el input por defecto en HTML5 <input type="date">.

Sobre el previsualizador creo que con un link a "previsualizar imagen" y que abra la imagen en una nueva pestaña seria suficiente.

Para las validaciones de formulario debería estar el componente Abide de Foundation 🤔

@raul-fuentes
Copy link
Contributor

Hola

Aquí os muestro un como creo que va a quedar el formulario.

  • formulario principal
    main_form

  • Declaración de actividades
    activities

  • Declaración de bienes patrimoniales
    assets

Aun tengo pendientes algunas cosas

  • Terminar las validaciones, tienen que ignorarse cuando no se rellenan campos en las pestañas de añadir en actividades o bienes patrimoniales
  • mejorar como se muestra la imagen seleccionada
  • gestionar correctamente en el controlador el rollback cuando se crea o actualiza un objeto
  • Testearlo todo

Indicadme, por favor, si es necesario hacer alguna modificacion

@decabeza
Copy link
Contributor

Hola @raul-fuentes a nivel de interfaz lo veo perfecto 😌 algún detalle menor como el tamaño de las pestañas que podremos corregir en futuras PRs (pero nada importante) 😉

@raul-fuentes
Copy link
Contributor

Hola

Tengo una duda, al mover los botones de aqui:

38022151-09d8eea2-327f-11e8-80b3-4bdfc2d54e73

a aqui:

screenshot from 2018-04-27 11-15-07

me parece que añadir el boton de editar facilitaria la tarea del admin (manteniendo el otro), porque actualmente ese boton solo esta disponible en la vista publica y lo veo enrevesado para acceder.

Ademas el formulario de edicion de personas te redirige (en caso de terminar de forma correcta) a la vista publica de la persona editada (funciona igual para nuevos registros) ¿no deberia de llevar al index de personas en admin (con una notificacion de proceso correcto, claro)?

Por lo demas la issue esta a falta de terminar los tests y escribir el PR

Un saludo

@decabeza
Copy link
Contributor

Hola @raul-fuentes

+1 a lo que comentas:

  • En la vista pública que solo aparezca el botón "Editar" para admins.

  • Añadir en esas acciones de admin el botón "Editar".

  • Que el formulario de editar (y de nuevos registros) al guardar lleve al index de esa sección de admin.

  • Extra: que el nombre en el index de admin sea un link a la vista pública.

😉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants