verdnatura/salix-front
verdnatura
/
salix-front
2
3
Fork 1
Code Issues Pull Requests 78 Releases Wiki Activity

refs #6694 VnLocation #154

Merged
jsegarra merged 31 commits from 6280_vnLocation into dev 2024-01-29 13:23:04 +00:00
Conversation 2 Commits 31 Files Changed 6 +49 -12
4 changed files with 49 additions and 12 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files
Showing only changes of commit 45dc9c355f - Show all commits

15
src/components/common/VnLocation.vue
View File

@ -54,11 +54,16 @@ const value = computed({
},
});
onMounted(() => {
locationFilter()
});
function setOptions(data) {
myOptions.value = JSON.parse(JSON.stringify(data));
myOptionsOriginal.value = JSON.parse(JSON.stringify(data));
}
setOptions(options.value);
watch(options, (newValue) => {
setOptions(newValue);
});
@ -67,17 +72,15 @@ function showLabel(data) {
return `${data.code} - ${data.town}(${data.province}), ${data.country}`;
}
onMounted(() => {
locationFilter()
})
function locationFilter(search) {
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-01-15 07:53:07 +00:00
Outdated
Review
Copy Link

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.
let args = { filter:{limit: 30}, search};
postcodesRef.value.fetch({args});
let where = { search };
postcodesRef.value.fetch({filter:{ where}, limit: 30});
}
function handleFetch( data) {
postcodesOptions.value = data;
}
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-01-15 08:37:07 +00:00
Outdated
Review
Copy Link

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)
jsegarra commented 2024-01-15 19:35:46 +00:00
Outdated
Review
Copy Link

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.
async function onPostcodeCreated(){
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-01-26 07:10:12 +00:00
Outdated
Review
Copy Link

Si solo se usa una vez, llamar directamente a locationFilter

Si solo se usa una vez, llamar directamente a locationFilter
jsegarra commented 2024-01-26 07:58:40 +00:00
Outdated
Review
Copy Link

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.
locationFilter()
};
jsegarra marked this conversation as resolved Outdated
jsegarra commented 2024-01-23 09:32:03 +00:00
Outdated
Review
Copy Link

Opcion A=> limit="30"

Opcion A=> limit="30"

7
src/pages/Customer/CustomerCreate.vue
View File

@ -33,9 +33,10 @@ const businessTypesOptions = ref([]);
const postcodesOptions = ref([]);
function handleLocation(data, { city, postcode, provinceFk, countryFk }) {
data.postcode = postcode;
data.city = city;
function handleLocation(data, location ) {
const { town, code, provinceFk, countryFk } = location ?? {}
data.postcode = code;
data.city = town;
data.provinceFk = provinceFk;
data.countryFk = countryFk;
}

7
src/pages/Worker/WorkerCreate.vue
View File

@ -57,9 +57,10 @@ const onBankEntityCreated = (data) => {
};
function handleLocation(data, { city, postcode, provinceFk, countryFk }) {
data.postcode = postcode;
data.city = city;
function handleLocation(data, location ) {
const { town, postcode: code, provinceFk, countryFk } = location ?? {}
data.postcode = code;
data.city = town;
data.provinceFk = provinceFk;
data.countryFk = countryFk;
}

32
test/cypress/integration/VnLocation.spec.js Normal file
View File

@ -0,0 +1,32 @@
const inputLocation = ':nth-child(3) > :nth-child(1) > .q-field > .q-field__inner > .q-field__control';
const locationOptions ='[role="listbox"] > div.q-virtual-scroll__content > .q-item'
describe('VnLocation', () => {
beforeEach(() => {
cy.viewport(1280, 720);
cy.login('developer');
cy.visit('/#/worker/create');
cy.waitForElement('.q-card');
});
// it('Show all options', function() {
jsegarra marked this conversation as resolved Outdated
alexm commented 2024-01-26 07:15:40 +00:00
Outdated
Review
Copy Link

En vez de comentarlo, quitarlo

En vez de comentarlo, quitarlo
alexm commented 2024-01-26 07:15:40 +00:00
Outdated
Review
Copy Link

En vez de comentarlo, quitarlo

En vez de comentarlo, quitarlo
alexm commented 2024-01-29 07:13:45 +00:00
Outdated
Review
Copy Link

❗En vez de comentarlo, quitarlo

❗En vez de comentarlo, quitarlo
// cy.get(inputLocation).click();
// cy.get(locationOptions).should('have.length',5)
// // cy.get(':nth-child(3) > :nth-child(1) > .q-field > .q-field__inner > .q-field__control > :nth-child(2) > .q-icon').click()
// });
// it('input filter location as "al"', function() {
// cy.get(inputLocation).click();
// cy.get(inputLocation).clear();
// cy.get(inputLocation).type('al');
// cy.get(locationOptions).should('have.length',3);
// });
it('input filter location as "ecuador"', function() {
cy.get(inputLocation).click();
cy.get(inputLocation).clear();
cy.get(inputLocation).type('ecuador');
cy.get(locationOptions).should('have.length',1);
cy.get(`${locationOptions}:nth-child(1)`).click();
cy.get(':nth-child(3) > :nth-child(1) > .q-field > .q-field__inner > .q-field__control > :nth-child(2) > .q-icon').click();
});
})