refs #6694 VnLocation #154
Labels
No Milestone
No Assignees
3 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#154
Loading…
Reference in New Issue
No description provided.
Delete Branch "6280_vnLocation"
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?
Se ha creado una tarea padre, #6694
WIP: #5888 #6280 VnLocationto #5888 #6280 VnLocation#5888 #6280 VnLocationto refs #6694 VnLocationLa rama esta incompleta.
Debería estar en WIP y sin revisores supongo
@ -0,0 +72,4 @@
</script>
<template>
No veo en que momento se hace el filtrado de datos a las tablas.
@ -222,2 +137,2 @@
v-model="data.countryFk"
/>
v-model="data.location"
@update:model-value="(data)=> console.log(data)"
Aqui deberia estar supongo
Vaya, parece que el último push no se había completado correctamente.
Muchas gracias
@ -0,0 +79,4 @@
@on-fetch="(data) => (postcodesOptions = data)"
auto-load
/>
<FetchData
De esta manera te estas descargando toda la tabla
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)
Tendré en cuenta estos datos para hacer unos cambios con respecto al código original.
Gracias por la aportación.
@ -0,0 +105,4 @@
map-options
use-input
:filter-options="['code','town.name']"
:filter-rules="['val.length>2']"
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
Si la consulta no es muy lenta/pesada yo no pondria nada.
Lo comentamos.
@ -0,0 +108,4 @@
:filter-rules="['val.length>2']"
clearable
hide-selected
fill-input
Si al final se fa la peticio en la back li possaria un delay (input-bounce) crec que se dia
Lo comentamos
corregido
655c821075
refs #6694 VnLocationto WIP: refs #6694 VnLocationWIP: refs #6694 VnLocationto refs #6694 VnLocation@ -0,0 +84,4 @@
ref="postcodesRef"
url="Postcodes/filter"
@on-fetch="(data) =>handleFetch(data)"
auto-load
Sin auto-load y hacer con codigo per con limit
O con auto-load pero con limit
El limit lo tienes definido en la línea 71, es correcto?
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)
@ -0,0 +95,4 @@
:placeholder="t('Search by postalCode, town, province or country')"
@input-value="locationFilter"
:default-filter="false"
:input-debounce="100"
Igual se lo pondria mas alto, ver como esta en salix
Corregido
6809e9bddb
@ -0,0 +1,14 @@
export async function deepFind(obj, path) {
Un includes no funcionaria igual?
No te sigo, pero lo comentamos si quieres sobre un ejemplo
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
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
Corregido
5d888a2977
@ -0,0 +83,4 @@
<FetchData
ref="postcodesRef"
url="Postcodes/filter"
@on-fetch="(data) =>handleFetch(data)"
Opcion A=> limit="30"
New commits pushed, approval review dismissed automatically according to repository settings
refs #6694 VnLocationto WIP: refs #6694 VnLocationFalta el test e2e
WIP: refs #6694 VnLocationto refs #6694 VnLocation@ -0,0 +81,4 @@
postcodesOptions.value = data;
}
async function onPostcodeCreated(){
Si solo se usa una vez, llamar directamente a locationFilter
Por mantener el formato en el uso de este componente.
Pero se puede borrar.
@ -0,0 +111,4 @@
fill-input
>
<template #form>
<CreateNewPostcode @on-data-saved="onPostcodeCreated($event)" />
Y tampoco veo que se este usando el evento no?
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
@ -13,3 +13,3 @@
},
optionLabel: {
type: String,
type: [String, Function],
No veo el pq hacerlo function
@ -17,3 +17,3 @@
},
filterOptions: {
type: Array,
type: [Array, Function],
No veo el pq hacerlo function
No veo el pq hacerlo function
@ -20,0 +19,4 @@
type: [Array, Function],
default: () => [],
},
filterRules: {
No veo que se use
@ -52,3 +57,4 @@
const search = val.toString().toLowerCase();
if (!search) return options;
if($props.filterRules.length) {
No veo que se use
@ -0,0 +8,4 @@
cy.waitForElement('.q-card');
});
// it('Show all options', function() {
En vez de comentarlo, quitarlo
En vez de comentarlo, quitarlo
❗En vez de comentarlo, quitarlo
New commits pushed, approval review dismissed automatically according to repository settings