Warmfix: #6943 addressPropagate #1389

Merged
jsegarra merged 9 commits from 6943_re_propagate into test 2025-02-17 12:20:34 +00:00
Member
No description provided.
jsegarra added 1 commit 2025-02-13 12:25:56 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
f821949740
feat: refs #6943 addressPropagate
jsegarra reviewed 2025-02-13 12:29:52 +00:00
@ -98,3 +98,3 @@
const emit = defineEmits(['onFetch', 'onDataSaved']);
const modelValue = computed(
() => $props.model ?? `formModel_${route?.meta?.title ?? route.name}`
() => $props.model ?? `formModel_${route?.meta?.title ?? route.name}`,
Author
Member

formato

formato
jsegarra reviewed 2025-02-13 12:30:06 +00:00
jsegarra reviewed 2025-02-13 12:30:15 +00:00
@ -223,3 +223,3 @@
if ($props.urlCreate) notify('globals.dataCreated', 'positive');
updateAndEmit('onDataSaved', formData.value, response?.data);
updateAndEmit('onDataSaved', formData.value, response?.data, originalData.value);
Author
Member

Añado el objeto old para comprobar desde quien lo instancia

Añado el objeto old para comprobar desde quien lo instancia
jsegarra added 3 commits 2025-02-14 01:43:50 +00:00
jsegarra added 1 commit 2025-02-14 16:18:40 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
fc80946817
fix: refs #6943 minor changes
jsegarra added 1 commit 2025-02-16 23:38:35 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
54b4791849
test: refs #6943 fix tests
jsegarra added 1 commit 2025-02-16 23:44:16 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
a3804d5e10
Merge branch 'test' into 6943_re_propagate
Author
Member

image
image

![image](/attachments/1421ede3-d606-480c-bf13-9a496ed5de53) ![image](/attachments/c8bbaf14-a126-41fa-a4a6-bc24fee3ece8)
jsegarra requested review from jgallego 2025-02-16 23:44:27 +00:00
jgallego requested changes 2025-02-17 06:12:18 +00:00
Dismissed
@ -33,0 +45,4 @@
quasar.dialog({
component: VnConfirm,
componentProps: {
title: t('You changed the equalization tax'),
Owner

esto es porque se olvidó migrar

esto es porque se olvidó migrar
Author
Member

exacto.

exacto.
@ -190,1 +223,4 @@
Daily invoice: Facturación diaria
Equivalent tax spreaded: Recargo de equivalencia propagado
You changed the equalization tax: Has cambiado el recargo de equivalencia
Do you want to spread the change?: ¿Deseas propagar el cambio a sus consignatarios?
Owner

en las claves no van signos

en las claves no van signos
Author
Member

El de ? si que se permite o lo hemos continuado
hay 38 resutlados para ?: ¿

El de ? si que se permite o lo hemos continuado hay 38 resutlados para `?: ¿`
@ -82,3 +82,3 @@
<VnInput
:label="t('Consignee')"
required
:required="true"
Owner

esto tal como estaba no hacia nada?

esto tal como estaba no hacia nada?
Author
Member

Parece que el composable había que darle una vuelta. Lo he simplificado y le he puesto un vitest

Parece que el composable había que darle una vuelta. Lo he simplificado y le he puesto un vitest
@ -13,0 +35,4 @@
.click();
});
cy.get(
'.q-card > :nth-child(1) > :nth-child(2) > .q-checkbox > .q-checkbox__inner',
Owner

aqui no aplica en cy?

aqui no aplica en cy?
Author
Member

podría aplicar pero en esta rama no está disponible Vncheckbox y habría que reemplazarlo
Quizás tarea de refactor a futuro, para practicas

podría aplicar pero en esta rama no está disponible Vncheckbox y habría que reemplazarlo Quizás tarea de refactor a futuro, para practicas
jsegarra added 1 commit 2025-02-17 11:18:48 +00:00
gitea/salix-front/pipeline/pr-test There was a failure building this commit Details
1d3be4856b
fix: refs #6943 required
jsegarra added 1 commit 2025-02-17 11:20:05 +00:00
gitea/salix-front/pipeline/pr-test This commit looks good Details
985eac702f
Merge branch 'test' into 6943_re_propagate
jsegarra requested review from jgallego 2025-02-17 11:20:09 +00:00
jgallego approved these changes 2025-02-17 11:30:21 +00:00
jsegarra merged commit 38df9c27ae into test 2025-02-17 12:20:34 +00:00
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 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#1389
No description provided.