#6772 - Reload data section when change Id #192
No reviewers
jgallego
Labels
No Milestone
No Assignees
4 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#192
Loading…
Reference in New Issue
No description provided.
Delete Branch "6772_reload_sections"
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: #6772 - Reload data section when change Idto #6772 - Reload data section when change IdNew commits pushed, approval review dismissed automatically according to repository settings
@ -89,0 +99,4 @@if (route.params.id) store.url = store.url.replace(/(\d+)/, route.params.id);} else {// Cuando el usuario cambia en la URLconst claveFk = Object.keys(arrayData.store.filter.where).find((clave) =>Clave seria Key?
Me puedes decir un sitio donde esto hace que pase de no ir a ir?
En ClaimPhoto mismo
Te busco otro
Si vas a https://dev-lilium.verdnatura.es/#/customer/34655/balance y cambias el ID a este https://dev-lilium.verdnatura.es/#/customer/4651/balancee, tienes que hacer F5
Lo decia pq me da la sensación de que FormModel si que refresca al cambiar de id. pq por debajo usa ArrayData
Igual si se lo añadimos a VnPaginate funciona igual.
Por centralizar funcionalidad
Por ejemplo en Claim. Basic data si que funciona al cambiar de ids
Mmm...en la tarea hay mas ejemplos, Entry/:id/buys por ejemplo
No lo veo, son componentes muy diferentes. Si quieres montamos reu para hablarlo
@ -86,7 +88,28 @@ watch(store.data = props.data;});watch(Esta parte del reload la veo mas en arrayData
New commits pushed, approval review dismissed automatically according to repository settings
He probado claimDevelopment y no funciona. Supongo que debería dado que usa CrudModel → VnPaginate → ArrayData. He puesto esto
:data-key="$props.url"en VnCrudModel en la parte del VnPaginate por si fuera eso pero tampoco.En ClaimPhoto si q funciona pero supongo que sera por el
watch)New commits pushed, approval review dismissed automatically according to repository settings
He resuelto el problema para la ventana de ClaimDevelopment
Me he asegurado que el resto de ventanas siga funcionando.
He modificado el checklist del redmine con los componentes reportados y que hay que probar antes de fusionar para validar que todo está OK
@ -36,0 +39,4 @@store.data = null;if (!store?.filter?.where) {//Cuando el cambio viene por VnSearchbarif (route.params.id) store.url = store.url.replace(/(\d+)/, route.params.id);Haciendo
return store.url = store.url.replace(/(\d+)/, route.params.id);te evitas el else y queda un poco mas legibleResuelto:
3359d5b6aeCorregir tests primero
@ -91,3 +93,3 @@});onBeforeRouteUpdate((to, from, next) => {Esta part entonces deuria estar igual pero en FormModel per exemple??
Si es així igual valdria la pena ficar-ho en ArrayData y en una prop poder habilitar-ho o deshabilitar-ho?
Mmm...el fallo no está en los formularios, ya que en FormModel no se han reportado errores.
Por otra parte, en que casos querrías deshabitar la funcionalidad de que al cambiar la URL cambie la ventana?
Mover la funcionalidad a ArrayData
la funcion fecth cambiar a fetch({append:false});
Añadir condición para decidir que casos no se quiere. Por defecto si se quiere
Probar en formModel
Lo he aplciado segun hemos comentado pero repite las peticiones al API.
Te dejo un patch
Lo he aplicado segun hemos comentado pero repite las llamadas al API
Te dejo un patch
En la tarea de VnTable
Añadiendo:
Ya funciona, lo dejamos asi? Seria añadirlo directamente en VnPaginate
https://gitea.verdnatura.es/verdnatura/salix-front/pulls/396/files#diff-20f84c64ff7b239be8a917fff2510bec3305dbfa
@alexm si, correcto, hay cambios que he propuesto que ya no son necesarios.
Sin embargo, claimPhoto sigue necesitando el código que hay en la PR
@ -38,6 +38,11 @@ vi.mock('vue-router', () => ({meta: { moduleName: 'mockName' },path: 'mockSection/list',}),onBeforeRouteUpdate: () => ({Esto hace falta?
@ -39,2 +39,4 @@path: 'mockSection/list',}),onBeforeRouteUpdate: () => ({to: 1,Este código no hace falta para que pasen los tests de front. Además esta puesto en el archivo global.
Quitar
@ -145,2 +145,4 @@dragFile.value = true;}watch(Incluiré este codigo en la v6 de https://redmine.verdnatura.es/issues/6336
vaaciones
vacaciones
no
New commits pushed, approval review dismissed automatically according to repository settings