hotFix(orderCatalogFilter): fix searchByTag #961
No reviewers
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#961
Loading…
Reference in New Issue
No description provided.
Delete Branch "hotFix_orderCatalogFilter_values"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
WIP: fix(orderCatalogFilter): fix searchByTagto fix(orderCatalogFilter): fix searchByTagCodigo subido y arreglado de https://gitea.verdnatura.es/verdnatura/salix-front/pulls/893/files
fix(orderCatalogFilter): fix searchByTagto hotFix(orderCatalogFilter): fix searchByTag@ -0,0 +2,4 @@
import { ref } from 'vue';
import { useI18n } from 'vue-i18n';
import axios from 'axios';
import VnSelect from 'components/common/VnSelect.vue';
src/
fallo mio
@ -0,0 +31,4 @@
emit('applyTags', tagInfo);
};
const removeTagGroupParam = (valIndex = null) => {
Me sale warning de no usarlo
@ -0,0 +57,4 @@
</script>
<template>
<QForm @submit="applyTags(tagValues)" class="all-pointer-events">
has definido el método pero sin argumentos, por tanto lo que le pases es irrelevante, no?
fallo mio
@ -0,0 +71,4 @@
rounded
:emit-value="false"
use-input
@update:model-value="($event) => getSelectedTagValues($event)"
@update:model-value="getSelectedTagValues" hace lo mismo no?
se me pasó
@ -0,0 +83,4 @@
v-if="!selectedTag?.isFree && tagOptions"
:label="t('components.itemsFilterPanel.value')"
v-model="value.value"
:options="tagOptions || []"
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
Se pasó
@ -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';
src/
se pasó
Igualmente hay un archivo jsconfig.json que sirve para los alias no veo el motivo de pq no usarlos
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
@ -13,3 +15,4 @@
const router = useRouter();
const stateStore = useStateStore();
const { t } = useI18n();
const arrayData = useArrayData('OrderCatalogList');
El parámetro aparece 4 veces, propuesta, de crear una constante, o desde el front llamar a arrayData.key
@ -57,3 +56,1 @@
}
};
const resetCategory = () => {
const resetCategory = (params, search) => {
Propuesta, ponerlo a continuación de quien la llama
fallo mio
@ -65,3 +59,1 @@
if (key === 'categoryFk') {
resetCategory();
}
params.categoryFk = null;
Veo que hay un par de sitios que hace params.category = null;
Propuesta: moverlo a una function resetParams, pej
Si wbuezas lo dejó asi
@ -146,0 +129,4 @@
}
const removeTagGroupParam = (search, valIndex) => {
if (!valIndex && valIndex !== 0) {
No importa la condición, siempre hará search
Sacarla del if...else
vaya, ni lo vi
@ -172,2 +169,4 @@
vnFilterPanelRef.value.search();
}
onMounted(() => {
En el OrderCatalog, hemos puesto los metodos de OnMounted, etc...arriba, aqui no?
se pasó