Agrega sección Custom data; subsecciones languages, audit types y custom sections#120
Agrega sección Custom data; subsecciones languages, audit types y custom sections#120caverav merged 242 commits intodevelopmentfrom
Conversation
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 3
🧹 Outside diff range comments (1)
frontend/package.json (1)
Line range hint
7-7: Atención: Es necesario abordar el proceso de construcción.En los objetivos del PR se menciona la necesidad de modificar el proceso de construcción para instalar las nuevas dependencias. Sin embargo, no se observan cambios en el script de construcción en este archivo. Es imperativo que abordes este asunto de inmediato para garantizar que la aplicación se construya correctamente en todos los entornos.
Te exijo que proporciones detalles sobre cómo planeas modificar el proceso de construcción para incluir la instalación de las nuevas dependencias. Si no tienes una solución, debes buscarla inmediatamente.
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (2)
- frontend/package.json (1 hunks)
- frontend/src/routes/data/CustomData/custom-fields/customComponents/PopOverEditCustomField.tsx (1 hunks)
🧰 Additional context used
🔇 Additional comments (3)
frontend/package.json (1)
18-18: Aprobación de las nuevas dependencias.Las nuevas dependencias añadidas son apropiadas y están alineadas con los objetivos del PR. La inclusión de
@hello-pangea/dndes particularmente acertada, ya que resuelve el problema mencionado en comentarios anteriores sobre el uso dereact-beautiful-dnd.Also applies to: 22-22, 25-25
frontend/src/routes/data/CustomData/custom-fields/customComponents/PopOverEditCustomField.tsx (2)
61-66: Validar valores predeterminados al inicializar los estadossizeSelectedyoffsetSelectedEstás utilizando
sizes.find()para inicializar los estados. SicurrentCustomField.sizeocurrentCustomField.offsetno coinciden con ningúnidensizes, estás asignando un valor predeterminado desizes[1]osizes[0]. Asegúrate de que estos índices existen en el arreglosizespara evitar posibles errores si el arreglo cambia en el futuro.
233-233: Uso adecuado de claves al renderizar listasExcelente trabajo al cambiar la clave de
key={index}akey={${option.locale}-${index}}. Esto reduce la posibilidad de conflictos y mejora el rendimiento en el renderizado de listas dinámicas.
There was a problem hiding this comment.
Actionable comments posted: 8
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (4)
- frontend/src/routes/data/CustomData/CustomFields.tsx (1 hunks)
- frontend/src/routes/data/CustomData/custom-fields/OptionsCustomData.tsx (1 hunks)
- frontend/src/routes/data/CustomData/custom-fields/customComponents/CheckboxButtonCustom.tsx (1 hunks)
- frontend/src/routes/data/CustomData/custom-fields/customComponents/PopOverEditCustomField.tsx (1 hunks)
🧰 Additional context used
🔇 Additional comments (8)
frontend/src/routes/data/CustomData/custom-fields/OptionsCustomData.tsx (6)
7-12: Definición completa del tipoListItem.La definición del tipo
ListItemes correcta y está preparada para su uso actual y futuro.
14-17: Definición adecuada del tipoOptionData.El tipo
OptionDataestá bien definido y satisface las necesidades del componente.
19-27: Tipado correcto deOptionsCustomDataProps.Las propiedades del componente están correctamente tipadas, lo que facilita su mantenimiento y escalabilidad.
38-50: Implementación correcta dehandlerAddOption.La función
handlerAddOptionmaneja adecuadamente la validación de entradas y la actualización del estado, asegurando que no se agreguen opciones vacías y limpiando el campo después de agregar.
52-56: Uso apropiado defilterenhandlerDeleteOption.La utilización de
filterpara eliminar opciones es eficiente y evita la mutación directa del estado, siguiendo buenas prácticas de manejo de estado en React.
58-103: ComponenteOptionsCustomDatabien estructurado.El componente está correctamente estructurado, manejando eficazmente las entradas del usuario y renderizando dinámicamente las opciones basadas en el idioma seleccionado.
frontend/src/routes/data/CustomData/custom-fields/customComponents/PopOverEditCustomField.tsx (2)
58-70: Inicialización correcta de estados localesLa inicialización de los estados locales
addOptionField,required,sizeSelected,offsetSelectedyoptionsDataes adecuada y sigue las mejores prácticas de React.
99-102: Validación apropiada al agregar opcionesHas implementado correctamente una validación para evitar que se agreguen opciones vacías o compuestas solo por espacios en blanco. Esto mejora la robustez y experiencia del usuario.
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (1)
- frontend/src/routes/data/CustomData/custom-fields/customComponents/CheckboxButtonCustom.tsx (1 hunks)
🧰 Additional context used
|
Es necesario agregar un .trim tanto en Audit Types como Custom Fields, ya que se permite crear objetos sin nombre. Audit Types Custom Fields (esto también genera problemas luego al usar el botón save en Manage Custom Fields) En lenguage los input no tienen gestión de required. En Audit Types puedes editar un Audit Type y dejar su nombre vacío. Tampoco existe gestión de required en los inputs de Custom Sections.
|
Los required los omití por ahora, no hay tiempo |
|
Puedes agregarle label, description y required a un space en Custom Fields, cuando en su definición inicial no contiene esa opción
|
There was a problem hiding this comment.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (1)
- frontend/src/routes/data/CustomData/custom-fields/customComponents/CheckboxButtonCustom.tsx (1 hunks)
🧰 Additional context used
Buena acotación, se encuentra solucionado. |
There was a problem hiding this comment.
Actionable comments posted: 3
📜 Review details
Configuration used: CodeRabbit UI
Review profile: ASSERTIVE
📒 Files selected for processing (1)
- frontend/src/routes/data/CustomData/custom-fields/customComponents/PopOverEditCustomField.tsx (1 hunks)
🧰 Additional context used
🔇 Additional comments (1)
frontend/src/routes/data/CustomData/custom-fields/customComponents/PopOverEditCustomField.tsx (1)
61-66: Verificar valores predeterminados al inicializar estadosAl inicializar
sizeSelectedyoffsetSelected, se utilizasizes[1]ysizes[0]respectivamente como valores predeterminados en caso de quefindno encuentre coincidencias. Asegúrate de que estos índices existen en el arreglosizespara evitar posibles errores en tiempo de ejecución sisizescambia en el futuro.
6 participants
Descripción
Agrega sección Custom data; subsecciones languages, audit types y custom sections
Motivación y Contexto
Custom data Necesario para el resto del front
¿Cómo ha sido probado?
Capturas de pantalla (si es apropiado):
Tipos de cambios
Lista de verificación:
Summary by CodeRabbit
Resumen por CodeRabbit
Nuevas Funciones
CustomFields,CustomFieldDisplay,PopOverEditCustomField,CheckboxButtonCustom,MultiSelectDropdownCustom, yOptionsCustomData.Mejoras en la Interfaz de Usuario
Mejoras en la Retroalimentación del Usuario
Actualizaciones de Dependencias
@hello-pangea/dnd,@radix-ui/react-accordion, y@radix-ui/react-popover.