Crea la vista para el buscador global de indicadores en monitoreo comunitario - #1064
Conversation
cgalvist
left a comment
cgalvist
left a comment
There was a problem hiding this comment.
Gracias @dothedada.
Desde mi punto de vista está bien y a nivel de interfaz de usuario no tuve ningún inconveniente.
| <Select value={filterYear} onValueChange={setFilterYear}> | ||
| <SelectTrigger | ||
| id="yearFilter" | ||
| className="bg-background [&_svg]:text-accent data-placeholder:text-foreground data-placeholder:texr-base" |
There was a problem hiding this comment.
acá hay un typo en texr-base
There was a problem hiding this comment.
corregido, aunque ahora me cuestiono si era necesario, jajajaja pues ni me di cuenta de que no se estana aplicando el texto
| ]); | ||
|
|
||
| useEffect(() => { | ||
| debouncedSearch((old) => ({ ...old, filter: searchFilter })); |
There was a problem hiding this comment.
Creo que acá falta la dependencia de debouncedSearch.
There was a problem hiding this comment.
hasta donde sé, no, pues es un ref y este en ningún momento se actualiza o cambia, es constante durante todo el ciclo de vida del componente (es mas, no debe cambar para que no putee el debounce), el eslint me gritaba ponerlo, pero me fui por lo sugerido por Erika de mantener los array de dependencias magros, entonces no lo puse. ya coloqué el ignore del eslint
3 participants
🛠️ Changes
Con este creo que ya se cierra la parte de indicadores* (el asterisco es porque se necesitan unos ajustes en el retorno del odata para mostrar la ubicación y el nombre de la iniciativa a la que está asociada el indicador).
El campo de búsqueda incluye tanto el nombre que se le de al indicador por la iniciativa como por el tipo de indicador. Los campos para filtrado por iniciativa o etiquetas funcionan con el odata, el de año con una lista en la configuracion y el de departamentos con la funcion
junto a esto, se creo una optimizacion del comboboxOdata, se creó una verision memoizada para que no dispare peticiones cada que hay un render, este se derivó del creado para el manejo de las etiquetas en los formularios.
Se actualizadoron los select y el manejo de los colores simplificados que se desarrollaron con los indicadores
📝 Associated issues
resolves LIB-822
🤔 Considerations
Aunque no debería haber problema cuando los ajustes del odata para indicadores se realicen pues ya contempla esos cambios, es necesario revisarla y confirmar que los filtros de búsqueda están ok, en especial los que hacen búsqueda dentro de arreglos, y pos limpiar las anottaciones que se dejaron para estos