verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 91 Releases Wiki Activity

fix: refs #7323 auto fill bic #642

Merged
jorgep merged 4 commits from 7323-hotfix-workerCreate into master 2024-08-26 11:57:11 +00:00
Conversation 0 Commits 4 Files Changed 3 +1 -1
1 changed files with 1 additions and 1 deletions
Showing only changes of commit e1e768948e - Show all commits
src/components/common
VnSelect.vue

2
src/components/common/VnSelect.vue
View File

@ -161,7 +161,7 @@ async function fetchFilter(val) {
if ($props.filterOptions.length) {
jorgep commented 2024-08-26 10:50:01 +00:00
Review

Permite filtrar por varias opciones. Antes, si era un digito, solo filtraba por id.

Permite filtrar por varias opciones. Antes, si era un digito, solo filtraba por id.
👍 1
defaultWhere = $props.filterOptions.reduce((obj, prop) => {
jorgep marked this conversation as resolved
jsegarra commented 2024-08-26 11:23:37 +00:00
Review

Porque la logica de $props.useLike no se puede usar en el reduce?

Porque la logica de $props.useLike no se puede usar en el reduce?
if (!obj.or) obj.or = [];
obj.or.push({ [prop]: { like: `%${val}%` } });
obj.or.push({ [prop]: $props.useLike ? { like: `%${val}%` } : val });
jorgep marked this conversation as resolved Outdated
jsegarra commented 2024-08-26 11:48:52 +00:00
Outdated
Review

Okey, pero ahora tenemos la misma logica 2 veces. La unica diferencia es como gestionas la variable key.
Porque no sacas el operador ternario a una funcion y luego lo asignas como valor a esa Key

Mas limpio, o que te parece?

Okey, pero ahora tenemos la misma logica 2 veces. La unica diferencia es como gestionas la variable key. Porque no sacas el operador ternario a una funcion y luego lo asignas como valor a esa Key Mas limpio, o que te parece?
return obj;
}, {});
} else