-
Notifications
You must be signed in to change notification settings - Fork 24
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
Comments
Hola, actualmente estan funcionanado estos enlaces, solo visibles si eres adminitrador ¿ los dejo asi o modifico su aspecto para que coincida con el mockup de @decabeza ? |
+1 a mantener el botón de editar y mover los demás a la zona admin |
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 |
De momento hay poca gente gestionado esta aplicación. Sólo admins 👍 |
@decabeza este es el aspecto actual del formulario de edicion/creacion de personas es un poco enorme para encajarlo en el mock, seria posible ponerlo como pestañas? |
@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" 😉 |
Hola! Por fin tengo algo que funciona, el hecho de que haya datos serializados esta siendo un dolor :( 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 :) Un saludo |
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 😌 |
Hola @raul-fuentes, Dos datos más que nos han pedido que son importantes poder modificar:
|
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 |
Hola @raul-fuentes, Para el datepicker puedes usar el input por defecto en HTML5 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 🤔 |
Hola Aquí os muestro un como creo que va a quedar el formulario. Aun tengo pendientes algunas cosas
Indicadme, por favor, si es necesario hacer alguna modificacion |
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) 😉 |
Hola Tengo una duda, al mover los botones de aqui: a aqui: 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 |
Hola @raul-fuentes +1 a lo que comentas:
😉 |
Qué
Los administradores deberían poder crear y editar perfiles individuales, incluidos los datos de sus declaraciones
Cómo
Mockups
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
The text was updated successfully, but these errors were encountered: