hotFix(orderCatalogFilter): fix searchByTag #961

Merged
alexm merged 8 commits from hotFix_orderCatalogFilter_values into master 2024-11-21 09:09:52 +00:00
Member
No description provided.
alexm added 1 commit 2024-11-18 14:06:30 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
c2db7be8cd
fix(orderCatalogFilter): fix searchByTag
alexm added 1 commit 2024-11-20 07:04:38 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e032d5988b
Merge branch 'master' into hotFix_orderCatalogFilter_values
alexm added 1 commit 2024-11-20 08:26:46 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
32fdc836f9
fix(OrderCatalogFilter): fix field value
alexm added 1 commit 2024-11-20 12:38:28 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
abd79283ff
fix(CatalogFilterValueDialog): from dev and fix
alexm changed title from WIP: fix(orderCatalogFilter): fix searchByTag to fix(orderCatalogFilter): fix searchByTag 2024-11-20 12:40:35 +00:00
alexm added 1 commit 2024-11-20 12:40:41 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
42d24359cd
Merge branch 'master' into hotFix_orderCatalogFilter_values
Author
Member
Codigo subido y arreglado de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/893/files
alexm changed title from fix(orderCatalogFilter): fix searchByTag to hotFix(orderCatalogFilter): fix searchByTag 2024-11-20 12:44:41 +00:00
jsegarra requested changes 2024-11-20 13:03:21 +00:00
Dismissed
@ -0,0 +2,4 @@
import { ref } from 'vue';
import { useI18n } from 'vue-i18n';
import axios from 'axios';
import VnSelect from 'components/common/VnSelect.vue';
Member

src/

src/
Member

fallo mio

fallo mio
alexm marked this conversation as resolved
@ -0,0 +31,4 @@
emit('applyTags', tagInfo);
};
const removeTagGroupParam = (valIndex = null) => {
Member

Me sale warning de no usarlo

Me sale warning de no usarlo
alexm marked this conversation as resolved
@ -0,0 +57,4 @@
</script>
<template>
<QForm @submit="applyTags(tagValues)" class="all-pointer-events">
Member

has definido el método pero sin argumentos, por tanto lo que le pases es irrelevante, no?

has definido el método pero sin argumentos, por tanto lo que le pases es irrelevante, no?
Member

fallo mio

fallo mio
alexm marked this conversation as resolved
@ -0,0 +71,4 @@
rounded
:emit-value="false"
use-input
@update:model-value="($event) => getSelectedTagValues($event)"
Member

@update:model-value="getSelectedTagValues" hace lo mismo no?

@update:model-value="getSelectedTagValues" hace lo mismo no?
Member

se me pasó

se me pasó
alexm marked this conversation as resolved
@ -0,0 +83,4 @@
v-if="!selectedTag?.isFree && tagOptions"
:label="t('components.itemsFilterPanel.value')"
v-model="value.value"
:options="tagOptions || []"
Member

En que punto tagOptions puede ser vacío? Por lo que veo tiene valor por defecto.

En todo caso seria al asignarle la variable data en la 55, pero si falla la petición no se asignaría valor

En que punto tagOptions puede ser vacío? Por lo que veo tiene valor por defecto. En todo caso seria al asignarle la variable data en la 55, pero si falla la petición no se asignaría valor
Member

Se pasó

Se pasó
alexm marked this conversation as resolved
@ -9,2 +9,4 @@
import OrderCatalogFilter from 'pages/Order/Card/OrderCatalogFilter.vue';
import VnSearchbar from 'src/components/ui/VnSearchbar.vue';
import getParamWhere from 'src/filters/getParamWhere';
import { useArrayData } from 'composables/useArrayData';
Member

src/

src/
Member

se pasó

se pasó
Author
Member

Igualmente hay un archivo jsconfig.json que sirve para los alias no veo el motivo de pq no usarlos

Igualmente hay un archivo jsconfig.json que sirve para los alias no veo el motivo de pq no usarlos
Member

He visto lo del alias, pero nadie los usa, quiero decir, por normal general el 80% de los archivos tienen la ruta completa
Así que creía que era el estandard.
Lo tengo en cuenta para asociarle el archivo a cypress

He visto lo del alias, pero nadie los usa, quiero decir, por normal general el 80% de los archivos tienen la ruta completa Así que creía que era el estandard. Lo tengo en cuenta para asociarle el archivo a cypress
alexm marked this conversation as resolved
@ -13,3 +15,4 @@
const router = useRouter();
const stateStore = useStateStore();
const { t } = useI18n();
const arrayData = useArrayData('OrderCatalogList');
Member

El parámetro aparece 4 veces, propuesta, de crear una constante, o desde el front llamar a arrayData.key

El parámetro aparece 4 veces, propuesta, de crear una constante, o desde el front llamar a arrayData.key
alexm marked this conversation as resolved
@ -57,3 +56,1 @@
}
};
const resetCategory = () => {
const resetCategory = (params, search) => {
Member

Propuesta, ponerlo a continuación de quien la llama

Propuesta, ponerlo a continuación de quien la llama
Member

fallo mio

fallo mio
alexm marked this conversation as resolved
@ -65,3 +59,1 @@
if (key === 'categoryFk') {
resetCategory();
}
params.categoryFk = null;
Member

Veo que hay un par de sitios que hace params.category = null;
Propuesta: moverlo a una function resetParams, pej

Veo que hay un par de sitios que hace params.category = null; Propuesta: moverlo a una function resetParams, pej
Member

Si wbuezas lo dejó asi

Si wbuezas lo dejó asi
alexm marked this conversation as resolved
@ -146,0 +129,4 @@
}
const removeTagGroupParam = (search, valIndex) => {
if (!valIndex && valIndex !== 0) {
Member

No importa la condición, siempre hará search
Sacarla del if...else

No importa la condición, siempre hará search Sacarla del if...else
Member

vaya, ni lo vi

vaya, ni lo vi
alexm marked this conversation as resolved
@ -172,2 +169,4 @@
vnFilterPanelRef.value.search();
}
onMounted(() => {
Member

En el OrderCatalog, hemos puesto los metodos de OnMounted, etc...arriba, aqui no?

En el OrderCatalog, hemos puesto los metodos de OnMounted, etc...arriba, aqui no?
Member

se pasó

se pasó
alexm marked this conversation as resolved
alexm added 1 commit 2024-11-21 06:42:08 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
c5acde22a3
chore: requested changes
alexm added 1 commit 2024-11-21 06:49:17 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
44f2ecca28
chore: requested changes
alexm requested review from jsegarra 2024-11-21 06:49:30 +00:00
jsegarra added 1 commit 2024-11-21 08:27:16 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
0434332ec4
Merge branch 'master' into hotFix_orderCatalogFilter_values
jsegarra approved these changes 2024-11-21 08:27:19 +00:00
alexm merged commit 0c3581f3dc into master 2024-11-21 09:09:52 +00:00
alexm deleted branch hotFix_orderCatalogFilter_values 2024-11-21 09:09:52 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: verdnatura/salix-front#961
No description provided.