Data UI required fixes - Githubissues

caverav / auditforge

AuditForge is a pentest reporting application making it simple and easy to write your findings and generate a customizable report.

https://auditforge.feriadesoftware.cl

MIT License

20 stars 0 forks source link

Data UI required fixes #67

Closed Sealra closed 2 months ago

Sealra commented 2 months ago

Descripción

Se agregan required en CRUD de Data, y si modifica el componente FileInput para que también sea capaz de gestionar required.

Motivación y Contexto

Mejora UI/UX

¿Cómo ha sido probado?

Usando npm run build, no afecta otras áreas.

Capturas de pantalla (si es apropiado):

Tipos de cambios

[ x ] Bugfix (cambio que no interrumpe el funcionamiento y que soluciona un problema)
[ ] New feature (cambio que no interrumpe el funcionamiento y que añade funcionalidad)
[ ] Breaking change (corrección o funcionalidad que podría causar que la funcionalidad existente cambie)

Lista de verificación:

[ x ] Mi código sigue el estilo de código de este proyecto.
[ ] Mi cambio requiere una modificación en la documentación.
[ ] He actualizado la documentación en consecuencia.

iTzGooDLife commented 2 months ago

Falta revisar si re repite en otros apartados.

Este comportamiento también ocurre en "Clients", "Companies" y "Templates".

Sealra commented 2 months ago

En las vistas de editar cliente no te obliga a rellenar los campos requeridos, por lo que se puede crear un cliente (rellenando los campos requeridos) y editarlo, pudiendo borrar los campos requeridos.

Corroborar si esto se repite en las demás secciones (collaborators, companies, templates) y corregir en ese caso.

Como comenté en la reunión, esto es gestionado en el backend, dando como resultado que si se edita un cliente (u otra sección) sin información en el modal o quitando la información de un campo requerido, este resultará en un success, manteniendo la data anterior, esto se podría evidenciar en un toast tal vez.

Sealra commented 2 months ago

Durante el proceso de ingresar un nuevo colaborador, si hay campos sin llenar y se hace el submit, entonces todos los campos requeridos se "vacian", en las imagenes se muestra más en detalle:

Se rellenan todos los campos excepto uno requerido:

Se hace el submit:

Falta revisar si re repite en otros apartados.

Esto será gestionado como comenté cuando se realice una gestion de errores específica para cada respuesta del backend, queda anotado !

iTzGooDLife commented 2 months ago

Esto será gestionado como comenté cuando se realice una gestion de errores específica para cada respuesta del backend, queda anotado !

Dices que esto se considera como un error del backend y su respuesta?

Sealra commented 2 months ago

Cerraré el PR dado que estaba pensado para mergear durante el día antes del sprint 1 para mostrar el tema de los required, aunque fuera de manera poco pulcra, volveré a realizar uno nuevo cuando haya una versión más definitiva !