refs #6694 VnLocation #154

Merged
jsegarra merged 31 commits from 6280_vnLocation into dev 2024-01-29 13:23:04 +00:00
Member

Se ha creado una tarea padre, #6694

Se ha creado una tarea padre, #6694
jsegarra added a new dependency 2024-01-11 13:01:19 +00:00
jsegarra added 4 commits 2024-01-12 10:28:23 +00:00
jsegarra changed title from WIP: #5888 #6280 VnLocation to #5888 #6280 VnLocation 2024-01-12 11:36:50 +00:00
jsegarra added 1 commit 2024-01-12 11:36:50 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
b6a03e6a9f
Merge branch 'dev' into 6280_vnLocation
jsegarra requested review from jgallego 2024-01-12 11:37:04 +00:00
jsegarra requested review from alexm 2024-01-12 11:37:05 +00:00
jsegarra changed title from #5888 #6280 VnLocation to refs #6694 VnLocation 2024-01-12 11:38:01 +00:00
alexm requested changes 2024-01-15 07:54:21 +00:00
alexm left a comment
Member

La rama esta incompleta.
Debería estar en WIP y sin revisores supongo

La rama esta incompleta. Debería estar en WIP y sin revisores supongo
@ -0,0 +72,4 @@
</script>
<template>
Member

No veo en que momento se hace el filtrado de datos a las tablas.

No veo en que momento se hace el filtrado de datos a las tablas.
jsegarra marked this conversation as resolved
@ -222,2 +137,2 @@
v-model="data.countryFk"
/>
v-model="data.location"
@update:model-value="(data)=> console.log(data)"
Member

Aqui deberia estar supongo

Aqui deberia estar supongo
jsegarra marked this conversation as resolved
jsegarra added 1 commit 2024-01-15 08:15:07 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
20835471a3
refs #6280 feat: handle option selected from VnLocation
Author
Member

La rama esta incompleta.
Debería estar en WIP y sin revisores supongo

Vaya, parece que el último push no se había completado correctamente.
Muchas gracias

> La rama esta incompleta. > Debería estar en WIP y sin revisores supongo Vaya, parece que el último push no se había completado correctamente. Muchas gracias
jsegarra added 1 commit 2024-01-15 08:22:26 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
84b53131c7
refs #6280 feat: deepFind as composable
jsegarra requested review from alexm 2024-01-15 08:22:35 +00:00
alexm requested changes 2024-01-15 08:42:04 +00:00
@ -0,0 +79,4 @@
@on-fetch="(data) => (postcodesOptions = data)"
auto-load
/>
<FetchData
Member

De esta manera te estas descargando toda la tabla

  • La tabla town tiene 71.691 registros.
  • La tabla postCode 271.259 registros.
  • ...

Por eso decia que no veia la funcion de filtrado. Yo no haría el filtrado en el front dado que te obliga a descargar todos los registros. Y en este caso lo veo inviable. (añadir limits tambien)

De esta manera te estas descargando toda la tabla - La tabla town tiene 71.691 registros. - La tabla postCode 271.259 registros. - ... Por eso decia que no veia la funcion de filtrado. Yo no haría el filtrado en el front dado que te obliga a descargar todos los registros. Y en este caso lo veo inviable. (añadir limits tambien)
Author
Member

Tendré en cuenta estos datos para hacer unos cambios con respecto al código original.
Gracias por la aportación.

Tendré en cuenta estos datos para hacer unos cambios con respecto al código original. Gracias por la aportación.
jsegarra marked this conversation as resolved
@ -0,0 +105,4 @@
map-options
use-input
:filter-options="['code','town.name']"
:filter-rules="['val.length>2']"
Member

Com veus esta part? @jgallego
Es una reglat de filtrat que t'obliga a escriure al menys 3 caracters per a que te filtre.
No ho acabe de vore. Pq si per exemple lo que volen es filtrar per pais pq hi ha un pais q saben q soles tenen 1 codi postal o molt pocs. i fiquen per exemple "J" (pq estan buscant Japon). no els trauria res

Com veus esta part? @jgallego Es una reglat de filtrat que t'obliga a escriure al menys 3 caracters per a que te filtre. No ho acabe de vore. Pq si per exemple lo que volen es filtrar per pais pq hi ha un pais q saben q soles tenen 1 codi postal o molt pocs. i fiquen per exemple "J" (pq estan buscant Japon). no els trauria res
Owner

Si la consulta no es muy lenta/pesada yo no pondria nada.

Si la consulta no es muy lenta/pesada yo no pondria nada.
Author
Member

Es una reglat de filtrat que t'obliga a escriure al menys 3 caracters per a que te filtre.
No ho acabe de vore. Pq si per exemple lo que volen es filtrar per pais pq hi ha un pais q saben q soles tenen 1 codi postal o molt pocs. i fiquen per exemple "J" (pq estan buscant Japon). no els trauria res

Lo comentamos.

> Es una reglat de filtrat que t'obliga a escriure al menys 3 caracters per a que te filtre. > No ho acabe de vore. Pq si per exemple lo que volen es filtrar per pais pq hi ha un pais q saben q soles tenen 1 codi postal o molt pocs. i fiquen per exemple "J" (pq estan buscant Japon). no els trauria res > Lo comentamos.
jsegarra marked this conversation as resolved
@ -0,0 +108,4 @@
:filter-rules="['val.length>2']"
clearable
hide-selected
fill-input
Member

Si al final se fa la peticio en la back li possaria un delay (input-bounce) crec que se dia

Si al final se fa la peticio en la back li possaria un delay (input-bounce) crec que se dia
Author
Member

Lo comentamos

Lo comentamos
Author
Member

corregido 655c821075

corregido 655c8210751f819ad4aff1480f94e57f73e22931
jsegarra marked this conversation as resolved
jsegarra changed title from refs #6694 VnLocation to WIP: refs #6694 VnLocation 2024-01-15 19:48:47 +00:00
jsegarra added 4 commits 2024-01-17 07:32:02 +00:00
jsegarra added 1 commit 2024-01-17 13:59:48 +00:00
jsegarra added 1 commit 2024-01-17 14:01:57 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
655c821075
refs #6694 feat input-debounce
jsegarra changed title from WIP: refs #6694 VnLocation to refs #6694 VnLocation 2024-01-17 14:02:12 +00:00
jsegarra requested review from alexm 2024-01-17 14:02:34 +00:00
jsegarra added 5 commits 2024-01-18 06:45:54 +00:00
jgallego approved these changes 2024-01-18 07:16:02 +00:00
Dismissed
alexm requested changes 2024-01-19 12:46:00 +00:00
@ -0,0 +84,4 @@
ref="postcodesRef"
url="Postcodes/filter"
@on-fetch="(data) =>handleFetch(data)"
auto-load
Member

Sin auto-load y hacer con codigo per con limit
O con auto-load pero con limit

Sin auto-load y hacer con codigo per con **limit** O con auto-load pero con limit
Author
Member

El limit lo tienes definido en la línea 71, es correcto?

El limit lo tienes definido en la línea 71, es correcto?
Member

Si y no. Eso sirve para las demás peticiones.
Pero no da más abrir la sección te carga toda la tabla (sin limit) al estar puesto el auto-load
(Lo he probado entrando en las ramas)

Si y no. Eso sirve para las demás peticiones. Pero no da más abrir la sección te carga toda la tabla (sin limit) al estar puesto el auto-load (Lo he probado entrando en las ramas)
jsegarra marked this conversation as resolved
@ -0,0 +95,4 @@
:placeholder="t('Search by postalCode, town, province or country')"
@input-value="locationFilter"
:default-filter="false"
:input-debounce="100"
Member

Igual se lo pondria mas alto, ver como esta en salix

Igual se lo pondria mas alto, ver como esta en salix
Author
Member

Corregido 6809e9bddb

Corregido 6809e9bddb37ba812c3e56a8f62f3beb255a051d
jsegarra marked this conversation as resolved
@ -0,0 +1,14 @@
export async function deepFind(obj, path) {
Member

Un includes no funcionaria igual?

Un includes no funcionaria igual?
Author
Member

No te sigo, pero lo comentamos si quieres sobre un ejemplo

No te sigo, pero lo comentamos si quieres sobre un ejemplo
Member

He revisado el codigo y no llego a ver cuando se usa eso ni tampoco veo el porque.
De hecho solo llegaria a usar en CustomerBasicData y tampoco pq se deberia quitar el filtro que tiene

He revisado el codigo y no llego a ver cuando se usa eso ni tampoco veo el porque. De hecho solo llegaria a usar en CustomerBasicData y tampoco pq se deberia quitar el filtro que tiene
Author
Member

Un ejemplo sería si quieres buscar por nombre de un pueblo en la petición de postocde, porque la estructura que devuelve postocde es
[
{ id: 1, town:{ id:1, name:'Algemesi'}}
]

Es verdad que esto se hizo para filtrar la carga inicial, pero con el número de resultados que manejamos, se descartó y por tanto este código también se debería.
Lo quito

Un ejemplo sería si quieres buscar por nombre de un pueblo en la petición de postocde, porque la estructura que devuelve postocde es [ { id: 1, town:{ id:1, name:'Algemesi'}} ] Es verdad que esto se hizo para filtrar la carga inicial, pero con el número de resultados que manejamos, se descartó y por tanto este código también se debería. Lo quito
Author
Member

Corregido 5d888a2977

Corregido 5d888a29778f04d264352cfae6d97f98f5442c4d
jsegarra marked this conversation as resolved
jsegarra reviewed 2024-01-23 09:32:03 +00:00
@ -0,0 +83,4 @@
<FetchData
ref="postcodesRef"
url="Postcodes/filter"
@on-fetch="(data) =>handleFetch(data)"
Author
Member

Opcion A=> limit="30"

Opcion A=> limit="30"
jsegarra marked this conversation as resolved
jsegarra added 3 commits 2024-01-23 10:36:09 +00:00
jsegarra dismissed jgallego’s review 2024-01-23 10:36:10 +00:00
Reason:

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

jsegarra added 1 commit 2024-01-23 10:37:28 +00:00
jsegarra added 1 commit 2024-01-23 11:53:50 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
37859892fa
refs #6280 perf: call locationFilter
jsegarra added 1 commit 2024-01-23 22:24:04 +00:00
jsegarra changed title from refs #6694 VnLocation to WIP: refs #6694 VnLocation 2024-01-23 22:52:00 +00:00
Author
Member

Falta el test e2e

Falta el test e2e
jsegarra added 1 commit 2024-01-24 07:30:35 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
45dc9c355f
refs #6280 perf: improve component
jsegarra changed title from WIP: refs #6694 VnLocation to refs #6694 VnLocation 2024-01-24 07:36:00 +00:00
jsegarra requested review from alexm 2024-01-24 07:36:42 +00:00
alexm requested changes 2024-01-26 07:15:44 +00:00
@ -0,0 +81,4 @@
postcodesOptions.value = data;
}
async function onPostcodeCreated(){
Member

Si solo se usa una vez, llamar directamente a locationFilter

Si solo se usa una vez, llamar directamente a locationFilter
Author
Member

Por mantener el formato en el uso de este componente.
Pero se puede borrar.

Por mantener el formato en el uso de este componente. Pero se puede borrar.
jsegarra marked this conversation as resolved
@ -0,0 +111,4 @@
fill-input
>
<template #form>
<CreateNewPostcode @on-data-saved="onPostcodeCreated($event)" />
Member

Y tampoco veo que se este usando el evento no?

Y tampoco veo que se este usando el evento no?
Author
Member

Por mantener el formato en el uso de este componente.
Diría que el valor de $event no se está gestionando de ninguna manera en el resto de instancias
En este caso se reemplaza por locationFilter() porque queremos los datos con otro formato

Por mantener el formato en el uso de este componente. Diría que el valor de $event no se está gestionando de ninguna manera en el resto de instancias En este caso se reemplaza por locationFilter() porque queremos los datos con otro formato
jsegarra marked this conversation as resolved
@ -13,3 +13,3 @@
},
optionLabel: {
type: String,
type: [String, Function],
Member

No veo el pq hacerlo function

No veo el pq hacerlo function
jsegarra marked this conversation as resolved
@ -17,3 +17,3 @@
},
filterOptions: {
type: Array,
type: [Array, Function],
Member

No veo el pq hacerlo function

No veo el pq hacerlo function
Member

No veo el pq hacerlo function

No veo el pq hacerlo function
jsegarra marked this conversation as resolved
@ -20,0 +19,4 @@
type: [Array, Function],
default: () => [],
},
filterRules: {
Member

No veo que se use

No veo que se use
jsegarra marked this conversation as resolved
@ -52,3 +57,4 @@
const search = val.toString().toLowerCase();
if (!search) return options;
if($props.filterRules.length) {
Member

No veo que se use

No veo que se use
jsegarra marked this conversation as resolved
@ -0,0 +8,4 @@
cy.waitForElement('.q-card');
});
// it('Show all options', function() {
Member

En vez de comentarlo, quitarlo

En vez de comentarlo, quitarlo
Member

En vez de comentarlo, quitarlo

En vez de comentarlo, quitarlo
Member

En vez de comentarlo, quitarlo

❗En vez de comentarlo, quitarlo
jsegarra marked this conversation as resolved
jsegarra added 1 commit 2024-01-26 07:56:46 +00:00
jsegarra added 1 commit 2024-01-26 08:04:11 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
c731b5b989
refs #6280: replace function event
jsegarra requested review from alexm 2024-01-26 08:06:36 +00:00
jsegarra added 1 commit 2024-01-29 07:16:45 +00:00
jsegarra added 1 commit 2024-01-29 07:20:46 +00:00
alexm approved these changes 2024-01-29 08:38:06 +00:00
Dismissed
jsegarra added 1 commit 2024-01-29 09:06:16 +00:00
jsegarra dismissed alexm’s review 2024-01-29 09:06:16 +00:00
Reason:

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

jsegarra added 1 commit 2024-01-29 13:14:27 +00:00
gitea/salix-front/pipeline/head This commit looks good Details
b0083544f0
Merge branch 'dev' into 6280_vnLocation
alexm approved these changes 2024-01-29 13:21:38 +00:00
jsegarra merged commit c5290339a1 into dev 2024-01-29 13:23:04 +00:00
jsegarra deleted branch 6280_vnLocation 2024-01-29 13:23:04 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
3 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#154
No description provided.