coderabbitai[bot]
commented
6 days ago 📝 Walkthrough
📝 Walkthrough
## Walkthrough Este pull request introduce varias modificaciones en el componente `Audits` ubicado en `frontend/src/routes/audits/audits.tsx`. Se actualiza la inicialización de la variable de estado `selectedValue`, estableciendo un valor predeterminado de '1'. Además, se modifica el mapeo de los datos de idioma, cambiando la propiedad `value` a `item.locale` y añadiendo una nueva propiedad `label`. También se mejora el manejo de errores en la función `handleSubmitNewAudit`, mostrando mensajes de error mediante notificaciones toast. Finalmente, se asegura que el estado de carga de idiomas se establezca en falso incluso si ocurre un error durante la carga. ## Changes | Archivos | Resumen de cambios | |-----------------------------------------------|---------------------------------------------------------------------------------------------------------| | `frontend/src/routes/audits/audits.tsx` | - Inicialización de `selectedValue` a '1'.- Modificación de mapeo de datos de idioma (`value` y `label`).
- Mejora en el manejo de errores en `handleSubmitNewAudit` con notificaciones toast.
- Actualización de `fetchData` para manejar el estado de carga en caso de error. | ## Possibly related PRs - **#138**: Las modificaciones en este PR implican cambios en el tipo `Finding` y la función `addVuln`, que pueden relacionarse con la funcionalidad general de auditoría que se está mejorando en el PR principal, particularmente en cómo se gestionan las auditorías y los hallazgos. - **#152**: Este PR actualiza las notificaciones toast para mensajes de éxito y error durante el proceso de guardado de auditorías, lo que se alinea con el enfoque del PR principal sobre el manejo de errores y la retroalimentación del usuario en la función `handleSubmitNewAudit`. - **#159**: Este PR introduce funcionalidad para exportar datos del panel de control, lo que puede conectar con la gestión general de auditorías y las mejoras en la interfaz de usuario discutidas en el PR principal, especialmente en términos de interacción del usuario y manejo de datos. ## Suggested labels `enhancement` ## Suggested reviewers - caverav - massi-ponce - Icksir - Sealra
📜 Recent review details
**Configuration used: CodeRabbit UI** **Review profile: CHILL**📥 Commits
Reviewing files that changed from the base of the PR and between 84ce4979da572dbf62bcde4976b879c14db17a5e and b11d0793f90d29a0449421b951c4f358caa1f504.📒 Files selected for processing (1)
* `frontend/src/routes/audits/audits.tsx` (4 hunks)🚧 Files skipped from review as they are similar to previous changes (1)
* frontend/src/routes/audits/audits.tsxThank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Descripción
Se realizan dos cambios:
Motivación y Contexto
El primer cambio es más que nada visual, ya que visualmente da la idea de que es opcional el seleccionar uno, cuando en realidad si no se selecciona uno, por defecto es "Default".
El segundo cambio arregla un bug existe en la app, ya que anteriormente se enviaba al backend el campo "language", y este no se reconocia en el dropdown al editar el audit, aunque los campos "auditType" y "language" tuvieran valores (si, pasaba a llevar a auditType). Con este cambio al agregar auditorias y posteriormente editarlas, aparecen los campos correspondientes.
Importante: Este cambio ayuda a las secciones de findings, debido a que anteriormente al agregar un finding se hacia utilizando el campo "language" y no "locale" de los lenguajes.
¿Cómo ha sido probado?
Para probar se debe añadir una auditoria con los campos necesarios y posteriormente ir a la edición de esta, se verá como se rellenan los dropdown correspondientes.
Capturas de pantalla (si es apropiado):
Valor por defecto
Antes:
Ahora:
Tipos de cambios
Lista de verificación:
Summary by CodeRabbit
Nuevas Funciones
Mejoras