HOTFIX #8217 Update Customer Credit #986
No reviewers
alexm
Labels
No Milestone
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies
No dependencies set.
Reference: verdnatura/salix-front#986
Loading…
Reference in New Issue
No description provided.
Delete Branch "hotfix_8217_updateCustomerCredit"
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: HOTFIX #8217 Update Customer Creditto HOTFIX #8217 Update Customer Credit@ -53,3 +54,3 @@onBeforeMount(() => {const { id } = route.params;getAddressesData(id);// getAddressesData(id);Comentado?
@ -127,2 +120,3 @@<div class="full-width flex justify-center"><QCard class="card-width q-pa-lg" v-if="addresses.length"><QCard class="card-width q-pa-lg"><VnPaginateAhora tiene un VnPaginate?
Me parece "rudimentario" dejar el codigo como estaba y preferia usar componentes nuestros.
Sin embargo, veo que el VnPagiante no tiene contenido, pero el resultado es el mismo
Voy a cambiarlo por FetchData
@ -34,3 +31,1 @@auto-loadmodel="customer"><FormModel :url-update="`Clients/${route.params.id}`" auto-load model="customer">En salix usaba :url="
Clients/${route.params.id}/getCard" dado que venia del cardPero el tema está en que CustomerDescriptor no muestra la misma info siempre. Segun la seccion en la que estes muestra o esconde iconos.
De esta manera no he encontrado fallos en el customer 1109
@ -93,22 +93,6 @@ const columns = computed(() => [<WorkerDescriptorProxy :id="row.worker.id" /></template></VnTable><!-- <QTableComentado?
Parece que es asi como estaba antes cuando se fusionó la #136 pero ahora se ha quitado
@ -39,1 +39,3 @@const setData = (entity) => (data.value = useCardDescription(entity?.name, entity?.id));const setData = (entity) => {data.value = useCardDescription(entity?.name, entity?.id);if (customer.value) customer.value.webAccess = data.value?.account?.isActive;Donde se use, usar
data.value?.account?.isActiveEn el icono noWeb,
Hay veces que no se sincroniza bien con el CustomerBasicData/CustomerFiscalData
Probar con el customer 1109
@ -34,7 +34,6 @@ function handleLocation(data, location) {/><FormModel:url-update="`Clients/${route.params.id}/updateFiscalData`":url="`Clients/${route.params.id}/getCard`"En salix usaba :url="
Clients/${route.params.id}/getCard" dado que venia del cardMisma situación que en el comentario anterior
"Pero el tema está en que CustomerDescriptor no muestra la misma info siempre. Segun la seccion en la que estes muestra o esconde iconos.
De esta manera no he encontrado fallos en el customer 1109"
@ -137,0 +45,4 @@ref="formModelRef"model="unpaid"url-update="ClientUnpaids":mapper="(formData) => ({ ...formData, clientFk: route.params.id })"formInitialData ?
La tienes una linea mas abajo
Dos dudas,
Pq se pasa de axios a fetchData si dijimos que era mejor usar axios?
Si el HOTfix es de customerCredit, hace falta añadir 42 archivos en el mismo PR que va directo a master?
Corregí los tests que fallaban, y me pareció mas correcto ponerles el nombre del modulo, por eso hay tantos archivos
NOTA: si ves que es mucho jaleo, puedo sacar el commit, corregimos funcionalidad y tests y en una rama desde dev, renombramos los tests
Con respecto al FetchData, no recuerdo ese criterio. Lo que recuerdo era intentar usar fetchData como composable. Mi objetivo fue que los componentes se pareciesen lo maximo entre ellos
Separalo mejor. Y los tests ya deberian estar yendo en dev
Adjunto el report
@ -93,22 +93,6 @@ const columns = computed(() => [<WorkerDescriptorProxy :id="row.worker.id" /></template></VnTable><!-- <QTableComentado?