#7136 - Enable paginate event in VnSelectFilter #255

Open
jsegarra wants to merge 86 commits from 7136_vnselectFilter_paginate into dev
Member
No description provided.
jsegarra changed title from #7136 - Enable paginate event in VnSelectFilter to WIP: #7136 - Enable paginate event in VnSelectFilter 2024-03-23 19:11:29 +00:00
jsegarra added 2 commits 2024-03-25 10:18:10 +00:00
jsegarra added 1 commit 2024-03-25 11:36:25 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b6f68321af
refs #7136 perf: arrayData
jsegarra added 1 commit 2024-03-25 11:36:59 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
b3a7574595
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra added 4 commits 2024-03-26 05:19:54 +00:00
jsegarra added 1 commit 2024-03-26 05:28:07 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
24483448dc
refs #7136 perf: handle when fetchRef not exists
jsegarra changed title from WIP: #7136 - Enable paginate event in VnSelectFilter to #7136 - Enable paginate event in VnSelectFilter 2024-03-26 12:45:18 +00:00
jsegarra added 1 commit 2024-03-26 12:45:21 +00:00
gitea/salix-front/pipeline/pr-dev Build queued... Details
d9200ca79f
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra requested review from alexm 2024-03-28 22:12:14 +00:00
jsegarra added 1 commit 2024-03-28 22:12:21 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
2ffd9538ba
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra added 3 commits 2024-03-28 23:42:18 +00:00
alexm requested changes 2024-04-02 07:36:58 +00:00
Dismissed
alexm left a comment
Member

Está mal enfocado.
Si equivale a , este si no se limita no se debería limitar por defecto.
Si se usa FetchData estas queriendo cargarte todos los datos normalmente.

En el caso de querer paginar en VnSelectFilter se debería paginar solo cuando te pasen la prop de url igual que funciona en salix(angularjs) ver ejemplo del propio InvoiceInFilter que has modificado modules/invoiceIn/front/search-panel/index.html L25.

Un caso perfecto para probar la funcionalidad (en Vue) sería en InvoiceInBasicData el VnSelectFilter de Suppliers.

(vn-crud-model si tiene la posibilidad de paginar pero es cuando se le define un vn-id="model" que luego lo usara el componente vn-data-viewer algo equivalente a nuestro VnPaginate pero no viene al caso)

Está mal enfocado. Si <FetchData> equivale a <vn-crud-model>, este si no se limita no se debería limitar por defecto. Si se usa FetchData estas queriendo cargarte todos los datos normalmente. En el caso de querer paginar en VnSelectFilter se debería paginar **solo** cuando te pasen la prop de **url** igual que funciona en salix(angularjs) ver ejemplo del propio InvoiceInFilter que has modificado `modules/invoiceIn/front/search-panel/index.html L25`. Un caso perfecto para probar la funcionalidad (en Vue) sería en InvoiceInBasicData el VnSelectFilter de Suppliers. _(`vn-crud-model` si tiene la posibilidad de paginar pero es cuando se le define un `vn-id="model"` que luego lo usara el componente `vn-data-viewer` algo equivalente a nuestro `VnPaginate` pero no viene al caso)_
Member

Aparte he probado el filtro de http://localhost:9000/#/invoice-in/list y va un poco raro. Si que pagina pero si lo haces rapido ya no, tienes que subir hacia arriba y luego volver a bajar y ahi si pagina

También había intentado buscarte más ejemplos de urls pero no me ha funcionado el regex en vscode VnSelectFilter[\s\S]*url

Aparte he probado el filtro de http://localhost:9000/#/invoice-in/list y va un poco raro. Si que pagina pero si lo haces rapido ya no, tienes que subir hacia arriba y luego volver a bajar y ahi si pagina También había intentado buscarte más ejemplos de urls pero no me ha funcionado el regex en vscode `VnSelectFilter[\s\S]*url`
jsegarra added 2 commits 2024-04-05 12:00:14 +00:00
Author
Member

@alexm

Te he dejado un commit para usar el paginate dentro de InvoiceInBasicData, para cuando se usa url dentro de VnSelectFilter.
Ya me dices.
PD: he intentado duplicar registros pero hay muchas restricciones en BD

Las regexp son complicadas pero saldrá jejejej

@alexm Te he dejado un commit para usar el paginate dentro de InvoiceInBasicData, para cuando se usa url dentro de VnSelectFilter. Ya me dices. PD: he intentado duplicar registros pero hay muchas restricciones en BD Las regexp son complicadas pero saldrá jejejej
jsegarra requested review from alexm 2024-04-10 17:51:57 +00:00
jsegarra added 1 commit 2024-04-10 17:52:02 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
7aa5292adc
Merge branch 'dev' into 7136_vnselectFilter_paginate
alexm requested changes 2024-04-12 06:31:01 +00:00
Dismissed
alexm left a comment
Member

No acaba de ir fino, cuando llegas al final. Sigues haciendo scroll y no carga nada mas. Y si haces scroll hacia arriba. Y vuelves a llegar al "final" ahí si que te carga más registros

No acaba de ir fino, cuando llegas al final. Sigues haciendo scroll y no carga nada mas. Y si haces scroll hacia arriba. Y vuelves a llegar al "final" ahí si que te carga más registros
jsegarra added 1 commit 2024-04-16 13:31:03 +00:00
gitea/salix-front/pipeline/pr-dev Build queued... Details
5b518eb2db
feat(lilium) #7136 remove Index
jsegarra requested review from alexm 2024-04-18 06:18:21 +00:00
jsegarra added 1 commit 2024-04-18 06:18:27 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
555299e8a7
Merge branch 'dev' into 7136_vnselectFilter_paginate
alexm requested changes 2024-04-18 09:40:36 +00:00
Dismissed
@ -50,3 +87,4 @@
if ($props.limit) filter.limit = $props.limit;
if ($props.skip) filter.skip = $props.skip;
const { data } = await axios.get($props.url, {
Member

Igual deberiamos sustiur el fetch por el fetch de arrayData y tambien al paginar usar el propio de arrayData
Asi duplicamos funcionalidad

Igual deberiamos sustiur el fetch por el fetch de arrayData y tambien al paginar usar el propio de arrayData Asi duplicamos funcionalidad
Author
Member

Revisado y refactorizado: 38e3e1c004

Revisado y refactorizado: 38e3e1c004f31ac26d439d4a74d765fd94d4a7c5
jsegarra added 3 commits 2024-04-18 10:24:44 +00:00
jsegarra requested review from alexm 2024-04-18 10:25:09 +00:00
jsegarra added 1 commit 2024-04-18 12:37:29 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
4580134416
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra added 1 commit 2024-04-19 06:42:48 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
688b6a9f9f
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra added 1 commit 2024-04-19 08:45:13 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
7640189f03
refs #7136 perf: remove fetchData from InvoiceInFilter
alexm requested changes 2024-04-22 06:04:51 +00:00
Dismissed
@ -33,8 +37,27 @@ const $props = defineProps({
},
});
const arrayData = useArrayData($props.url ?? $props.dataKey, {
Member

No estoy segur de si deberiamos poner la funcionalidad de usar arrayData//paginar en FechData si luego quien lo usa es VnSelectFilter, en vez de ponerla directamente en VnSelectFilter. @juan

No estoy segur de si deberiamos poner la funcionalidad de usar arrayData//paginar en FechData si luego quien lo usa es VnSelectFilter, en vez de ponerla directamente en VnSelectFilter. @juan
jsegarra added 3 commits 2024-05-16 10:07:11 +00:00
jsegarra added 4 commits 2024-05-27 07:16:31 +00:00
jsegarra added 5 commits 2024-05-27 09:54:19 +00:00
jsegarra added 1 commit 2024-05-27 13:00:16 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
cf67a0dc46
test: fix
jsegarra added 1 commit 2024-05-27 13:01:22 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
14e8c27997
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra requested review from alexm 2024-05-27 13:01:23 +00:00
alexm requested changes 2024-05-28 10:59:41 +00:00
Dismissed
@ -112,6 +112,8 @@ export function useArrayData(key, userOptions) {
return response;
}
function isHasMoreData() {}
Member

Creo que se te ha olvidado quitarla

Creo que se te ha olvidado quitarla
jsegarra marked this conversation as resolved
@ -51,3 +43,4 @@
<QItemSection>
<VnSelect
:label="t('Client')"
:options="clients"
Member

ya no hace falta el options, de hecho si arriba hay refs para los options tampoco hacen falta

ya no hace falta el options, de hecho si arriba hay refs para los options tampoco hacen falta
Author
Member

Gacias por el detalle, he visto que salespersons y authos también habia que quitar

Gacias por el detalle, he visto que salespersons y authos también habia que quitar
@ -50,3 +36,4 @@
dense
outlined
rounded
url="Suppliers"
Member

Nada mas entrar en http://localhost:9000/#/invoice-in/1/summary?limit=10 me falla y creo que esta peticion

Nada mas entrar en http://localhost:9000/#/invoice-in/1/summary?limit=10 me falla y creo que esta peticion
Member

image

![image](/attachments/00b698b4-86bb-47c9-a84f-15f853c38eaf)
7.5 KiB
Member

image

![image](/attachments/f9c463ea-0a0e-4fab-96e5-3a4fd21313c1)
5.0 KiB
jsegarra added 2 commits 2024-05-28 11:08:16 +00:00
jsegarra added 1 commit 2024-05-28 11:13:33 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
31d2ad6643
fix: #7136 default where value as object
Author
Member

image

He modificado el valor default de la prop where

> ![image](/attachments/f9c463ea-0a0e-4fab-96e5-3a4fd21313c1) He modificado el valor default de la prop where
jsegarra requested review from alexm 2024-05-28 11:14:41 +00:00
jsegarra added 5 commits 2024-05-29 09:06:34 +00:00
jsegarra added 2 commits 2024-05-29 10:31:46 +00:00
alexm requested changes 2024-06-07 05:39:04 +00:00
Dismissed
@ -65,11 +66,12 @@ const requiredFieldRule = (val) => val ?? t('globals.fieldRequired');
const { optionLabel, optionValue, optionFilter, options, modelValue } = toRefs($props);
const myOptions = ref([]);
const myOptionsFiltered = ref([]);
Member

Por como estaba creo recordar que myOptions era la variable para las opciones filtradas y myOptionsOriginal para las originales. myOptionsFiltered para que seria? Puede ser myOptions ?

Por como estaba creo recordar que `myOptions` era la variable para las opciones filtradas y `myOptionsOriginal ` para las originales. `myOptionsFiltered ` para que seria? Puede ser `myOptions ` ?
Author
Member

Me he dado cuenta que la necesidad de esa variable no cubre otro caso de uso asi que la elimino

Me he dado cuenta que la necesidad de esa variable no cubre otro caso de uso asi que la elimino
@ -88,0 +103,4 @@
setOptions(data);
return;
}
if ($props.options) setOptions($props.options);
Member

De primeras diria que esta linea no hace falta, dado que la primera ja lo hace

De primeras diria que esta linea no hace falta, dado que la primera ja lo hace
Author
Member

Pero quiero evitar que haga fetch
Un caso de uso sería: se lo declaro en el componente o una variable

Pero quiero evitar que haga fetch Un caso de uso sería: se lo declaro en el componente o una variable
jsegarra marked this conversation as resolved
@ -88,0 +104,4 @@
return;
}
if ($props.options) setOptions($props.options);
else fetchFilter($props.modelValue);
Member

Esta parte solo debe hacerla si hay url. Y por como esta ahora ya habria hecho la peticion al entrar en if (useURL.value)

Esta parte solo debe hacerla si hay url. Y por como esta ahora ya habria hecho la peticion al entrar en `if (useURL.value)`
Author
Member

Lo cambio para dejarlo asi
if (!$props.options) fetchFilter($props.modelValue);

Lo cambio para dejarlo asi if (!$props.options) fetchFilter($props.modelValue);
@ -119,0 +148,4 @@
}
async function fetchFilter(val) {
const { fields, sortBy, limit } = $props;
const where = buildwhere(val);
Member

Si de por si solo entra en fetchFilter si hay url, no hace falta separar la construccion del where en una funcion apart, que solo se usa en un sitio

Si de por si solo entra en fetchFilter si hay url, no hace falta separar la construccion del where en una funcion apart, que solo se usa en un sitio
Author
Member

Vale, lo debí separar para otra situación que acabé borrando y esto se quedó así
Lo limipio

Vale, lo debí separar para otra situación que acabé borrando y esto se quedó así Lo limipio
@ -150,3 +200,4 @@
<template>
<FetchData
v-if="useURL"
Member

Usando arrayData + lo que hablamos ayer creo que ya no hace falta el FetchData

Usando arrayData + lo que hablamos ayer creo que ya no hace falta el FetchData
jsegarra added 2 commits 2024-06-07 10:59:38 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
117c2e90db
comments
jsegarra added 2 commits 2024-06-07 14:57:05 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
464cf47a39
useFetchData composable
jsegarra changed title from #7136 - Enable paginate event in VnSelectFilter to WIP: #7136 - Enable paginate event in VnSelectFilter 2024-06-07 21:02:47 +00:00
jsegarra added 1 commit 2024-06-07 21:08:22 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
a23eb0a5ad
approach
jsegarra added 2 commits 2024-06-10 10:10:19 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
7292dca197
feat: remove fetchData to nativeCmp
jsegarra added 2 commits 2024-06-18 08:48:38 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
e775d922e8
perf: cleancode
jsegarra added 1 commit 2024-06-18 10:59:51 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
817fcc4485
feat: useArrayData
Author
Member

Se ha implementado el uso de ArrayData

Se ha implementado el uso de ArrayData
jsegarra changed title from WIP: #7136 - Enable paginate event in VnSelectFilter to #7136 - Enable paginate event in VnSelectFilter 2024-06-18 11:22:14 +00:00
jsegarra requested review from alexm 2024-06-18 11:22:17 +00:00
jsegarra added 1 commit 2024-06-18 11:22:22 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
9718f6f7b0
Merge branch 'dev' into 7136_vnselectFilter_paginate
alexm requested changes 2024-06-25 10:11:07 +00:00
Dismissed
@ -5,2 +5,3 @@
import FetchData from 'src/components/FetchData.vue';
const emit = defineEmits(['update:modelValue', 'update:options']);
import { useArrayData } from 'src/composables/useArrayData';
import { useFetchData } from 'src/composables/useFetchData';
Member

Quitar

Quitar
jsegarra marked this conversation as resolved
@ -84,2 +84,3 @@
if (append) myOptionsOriginal.value = JSON.parse(JSON.stringify(data));
}
onMounted(() => {
const useURL = computed(() => $props.url?.length > 0);
Member

Esto no sería lo mismo que hacerle un computed a $props.url ¿?

const useURL = computed(() => $props.url);

Esto no sería lo mismo que hacerle un computed a $props.url ¿? `const useURL = computed(() => $props.url);`
jsegarra marked this conversation as resolved
@ -86,0 +95,4 @@
limit: $props.limit,
sortBy: $props.sortBy,
fields: $props.fields,
} ?? {};
Member

Si lo estas definiendo no puede ser null no?

Si lo estas definiendo no puede ser null no?
jsegarra marked this conversation as resolved
@ -88,0 +103,4 @@
if (useURL.value) {
arrayData.store.userFilter = $props.where;
arrayData.store.filter.where = $props.where;
const { data } = await arrayData.fetch({ append: true });
Member

updateRouter: false sino podrá cosas en la url

`updateRouter: false` sino podrá cosas en la url
jsegarra marked this conversation as resolved
@ -117,3 +145,3 @@
const where = { ...{ [key]: { like: `%${val}%` } }, ...$props.where };
return dataRef.value.fetch({ fields, where, order: sortBy, limit });
return arrayData.value.fetch({ fields, where, order: sortBy, limit });
Member

Por como esta useArrayData, fetch no acepta los datos por paramtros. Solo { append = false, updateRouter = true }

Por como esta useArrayData, fetch no acepta los datos por paramtros. Solo `{ append = false, updateRouter = true }`
jsegarra marked this conversation as resolved
@ -0,0 +1,30 @@
// import { ref } from 'vue';
Member

Este archivo no se usar, quitarlo

Este archivo no se usar, quitarlo
jsegarra marked this conversation as resolved
jsegarra added 3 commits 2024-06-26 11:20:16 +00:00
jsegarra added 3 commits 2024-06-27 08:30:02 +00:00
jsegarra requested review from alexm 2024-06-27 08:30:44 +00:00
Author
Member

He resuelto un caso de uso que evitaba paginar después de haber escrito un valor

He resuelto un caso de uso que evitaba paginar después de haber escrito un valor
alexm reviewed 2024-07-05 05:48:14 +00:00
@ -89,0 +91,4 @@
const useURL = computed(() => $props.url);
const $attrs = useAttrs();
const arrayDataKey = $props.dataKey ?? ($props.url !== '' ? $props.url : $attrs.label);
Member

'' es false
$props.url ? $props.url : $attrs.label

'' es false `$props.url ? $props.url : $attrs.label`
jsegarra added 3 commits 2024-07-05 19:01:27 +00:00
alexm requested changes 2024-07-08 05:37:31 +00:00
Dismissed
alexm left a comment
Member

Otra cosa, me ha costado bastante de leer pq hay variables por el medio, watchers , el onMounted a mitad y tal.
Me lo he ordenado como (que es mas o menos como se recomienda, https://vuejs.org/style-guide/rules-recommended) :
-Variables
-Variables computed
-onMounted
-Watchers
-Funciones

Y me ha resultado mas fácil de leer

Otra cosa, me ha costado bastante de leer pq hay variables por el medio, watchers , el onMounted a mitad y tal. Me lo he ordenado como (que es mas o menos como se recomienda, https://vuejs.org/style-guide/rules-recommended) : -Variables -Variables computed -onMounted -Watchers -Funciones Y me ha resultado mas fácil de leer
@ -122,4 +164,1 @@
if (new RegExp(/\d/g).test(val)) key = optionValue.value;
const defaultWhere = $props.useLike
Member

El useLike no me lo quites pls

El useLike no me lo quites pls
Author
Member

hecho

hecho
@ -93,2 +114,3 @@
if ($props.focusOnMount) setTimeout(() => vnSelectRef.value.showPopup(), 300);
setOptions(options.value);
if ($props.url && $props.modelValue) fetchFilter($props.modelValue);
if (!$props.options) fetchFilter($props.modelValue);
Member

Que esten estas dos funciones a la vez lo veo raro (fetchFilter, initSelect)
Pq si no hay options y hay URL hara la peticion 2 veces. Yo diria que la funcion de initSelect no hace falta.

De hecho yo creo que solo deberia haber un fetch y cuando se quiera hacer un fetch se llame ahi (fetchFilter)

Que esten estas dos funciones a la vez lo veo raro (fetchFilter, initSelect) Pq si no hay options y hay URL hara la peticion 2 veces. Yo diria que la funcion de initSelect no hace falta. De hecho yo creo que solo deberia haber un fetch y cuando se quiera hacer un fetch se llame ahi (fetchFilter)
Author
Member

Creía que habían mas ocurrencias de initselect, lo he eliminado

Creía que habían mas ocurrencias de initselect, lo he eliminado
@ -133,1 +170,3 @@
let isFiltered = false;
watch(filterValue, (newVal, oldVal) => {
// if (vnSelectRef.value.modelValue) {
Member

Quitar

Quitar
jsegarra marked this conversation as resolved
@ -0,0 +9,4 @@
if (props.limit) filter.limit = props.limit;
return JSON.stringify(filter);
};
const fetch = async (filter) =>
Member

get no?

get no?
Author
Member

Lo he eliminado de la PR, y un ejemplo de su implementacion tambien.

Asi se queda lo mas limpio posible.
cuando surja la tarea si hace falta hacemos cherry-pick

Lo he eliminado de la PR, y un ejemplo de su implementacion tambien. Asi se queda lo mas limpio posible. cuando surja la tarea si hace falta hacemos cherry-pick
@ -127,3 +96,3 @@
</VnRow>
<VnRow class="row q-gutter-md q-mb-md">
<QSelect
<VnSelect
Member

He probado este y se vuelve loco el descriptor

He probado este y se vuelve loco el descriptor
Author
Member

Me tendrás que mostrar, please

Me tendrás que mostrar, please
Member

En general diría de revisar el código de VnSelect, pq creo que se han ido añadiendo funciones y se puede simplificar

En general diría de revisar el código de VnSelect, pq creo que se han ido añadiendo funciones y se puede simplificar
jsegarra added 1 commit 2024-07-12 10:38:51 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
217b26fbe8
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra added 1 commit 2024-07-12 10:49:58 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
cdee2155b2
perf: remove unnecessay code
jsegarra added 1 commit 2024-07-12 10:55:25 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
b00f955913
perf: vue-recomendations
jsegarra requested review from alexm 2024-07-12 10:56:14 +00:00
alexm requested changes 2024-07-18 06:20:34 +00:00
Dismissed
alexm left a comment
Member

Ver #551

Ver https://gitea.verdnatura.es/verdnatura/salix-front/pulls/551
jsegarra added 2 commits 2024-07-22 14:23:05 +00:00
jsegarra requested review from alexm 2024-07-22 14:23:23 +00:00
alexm approved these changes 2024-08-16 13:52:19 +00:00
Dismissed
jsegarra added 2 commits 2024-09-12 23:13:12 +00:00
jsegarra dismissed alexm’s review 2024-09-12 23:13:12 +00:00
Reason:

New commits pushed, approval review dismissed automatically according to repository settings

jsegarra requested review from alexm 2024-09-12 23:13:45 +00:00
alexm requested changes 2024-09-13 06:22:31 +00:00
alexm left a comment
Member

Revisar skip, deberia ser multiplo del limit
Tambien quitar codigo comentado

Revisar skip, deberia ser multiplo del limit Tambien quitar codigo comentado
jsegarra added 1 commit 2024-09-13 08:35:44 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
23faa707bf
Merge branch 'dev' into 7136_vnselectFilter_paginate
jsegarra added 2 commits 2024-09-13 08:59:00 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
ffed7125d0
perf: remove first call
jsegarra added 2 commits 2024-09-13 09:19:09 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
88e5dd08b8
perf: use $props.params
jsegarra added 2 commits 2024-09-13 19:31:51 +00:00
gitea/salix-front/pipeline/pr-dev There was a failure building this commit Details
82da5c2228
fix: solve cancelledSignal
jsegarra added 1 commit 2024-09-13 20:19:46 +00:00
gitea/salix-front/pipeline/pr-dev This commit looks good Details
0a86b27f53
test: improve test
jsegarra requested review from alexm 2024-09-13 20:19:59 +00:00
jsegarra added 2 commits 2024-10-11 13:24:14 +00:00
All checks were successful
gitea/salix-front/pipeline/pr-dev This commit looks good
Required
Details
This pull request has changes conflicting with the target branch.
  • src/components/common/VnSelect.vue
  • test/cypress/integration/entry/myEntry.spec.js
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#255
No description provided.