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

1 stars 0 forks source link

Data UI required fixes #94

Open Sealra opened 1 week ago

Sealra commented 1 week ago

Descripción

Se agregan required en CRUD de Data, y se modifica el componente FileInput para que también sea capaz de gestionar required. Además, se arregla comportamiento buggy en Clients.

Destacar que no soluciona problemas de ESLint warns del issue #52

Motivación y Contexto

Mejora UI/UX, soluciona issue #63

¿Cómo ha sido probado?

A través del uso de los modales, no afecta otras áreas.

Capturas de pantalla (si es apropiado):

Tipos de cambios

[ ] Bugfix (cambio que no interrumpe el funcionamiento y que soluciona un problema)
[x] 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.
[ ] Requiere nuevos tests.

iTzGooDLife commented 1 week ago

No se verifican los campos requeridos durante la creación del cliente. En la primera imagen se ven los campos requeridos, pero al editar, estos ya no lo son (segunda imagen) y en la tercera imagen se puede ver que los cambios se guardan aunque no se tengan esos campos (excepto el email). Lo dejé como comentario y no como cambios solicitados pq el backend permite el cambio, asi que se debería conversar que hacer en tal situación.

iTzGooDLife commented 1 week ago

No permite eliminar Clientes ni Compañias, no sé si es por que no está actualizado con la rama development o hay otros endpoints rotos con la actualización de dependencias. Revisar @Sealra @caverav

iTzGooDLife commented 1 week ago

No permite eliminar Clientes ni Compañias, no sé si es por que no está actualizado con la rama development o hay otros endpoints rotos con la actualización de dependencias.

Está actualizado, hay que arreglar eso. Se hará issue.

iTzGooDLife commented 1 week ago

Está actualizado, hay que arreglar eso. Se hará issue.

El problema era que development estaba desactualizado en comparación con main, el problema ya no está presente.

Sealra commented 1 week ago

No se verifican los campos requeridos durante la creación del cliente. En la primera imagen se ven los campos requeridos, pero al editar, estos ya no lo son (segunda imagen) y en la tercera imagen se puede ver que los cambios se guardan aunque no se tengan esos campos (excepto el email). Lo dejé como comentario y no como cambios solicitados pq el backend permite el cambio, asi que se debería conversar que hacer en tal situación.

En efecto, únicamente gestioné los errores del back en el caso de editar, dado que el resto de acciones sí era posible, podemos discutir en la próxima daily el comportamiento deseado.

Sealra commented 1 week ago

Pedí un nuevo review @iTzGooDLife dado que se corrigió el Regex !

massi-ponce commented 1 week ago

No se verifican los campos requeridos durante la creación del cliente. En la primera imagen se ven los campos requeridos, pero al editar, estos ya no lo son (segunda imagen) y en la tercera imagen se puede ver que los cambios se guardan aunque no se tengan esos campos (excepto el email). Lo dejé como comentario y no como cambios solicitados pq el backend permite el cambio, asi que se debería conversar que hacer en tal situación.

Pasa exactamente lo mismo con Collaborators. Primero cree uno, y luego lo edité limpiando todos los campos que antes eran required, y me dejó hacerlo. Adjunto fotos:

imagen

massi-ponce commented 1 week ago

Encontré otro "bug" en Clients pero no sé que tanta importancia deberíamos darle ya que igual es un poco rebuscado y, además, está muy relacionado a los ya comentados más arriba.

Pero básicamente se pueden pasar por alto los campos requeridos incluso a partir desde la creación del cliente como tal, es decir, no es necesario crear uno, luego editarlo y limpiar los campos como se indica en comentarios más arriba.

Adjunto fotos de casos, donde la primera imagen muestra como, al rellenar los campos requeridos con un espacio (menos el correo), permite crear un cliente "vacío" pasando por alto los campos requeridos. Esto se puede ver reflejado en la segunda imagen.

imagen imagen

Intenté replicar lo mismo en Collaborators pero, por alguna razón, ahí no lo permite y sale el siguiente mensaje en el toast:

imagen