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

#6943 Customer FiscalData already exists #1365

Closed
jsegarra wants to merge 7 commits from 6943_customerFD_alreadyExists into dev
Conversation 1 Commits 7 Files Changed 2 +61 -7
jsegarra commented 2025-02-10 12:43:08 +00:00
Member
No description provided.
jsegarra added 1 commit 2025-02-10 12:43:08 +00:00
jsegarra added 1 commit 2025-02-12 08:49:48 +00:00
jsegarra requested review from jgallego 2025-03-01 01:30:57 +00:00
jgallego requested changes 2025-03-01 07:19:19 +00:00
src/pages/Customer/Card/CustomerFiscalData.vue Outdated
@ -27,0 +37,4 @@
else await confirm(data);
}
async function checkExistingClient({ email, phone, mobile, id }) {
jgallego commented 2025-03-01 07:19:10 +00:00
Owner

que @alexm confirme técnicamente el enfoque i en caso que este correcto, hazle test de front.

que @alexm confirme técnicamente el enfoque i en caso que este correcto, hazle test de front.
jsegarra commented 2025-03-01 11:27:31 +00:00
Author
Member

Este caso es muy concreto por eso la nueva prop en FormModel
No puede usarse lo que hay porque al darle a guardar tiene que hacer una comprobación previa(beforeSave), y según eso ejecutar una cosa después del guardado onDataSaved.
Me parece complicado y fácil de entender, el separar un flujo en dos funciones

Este caso es muy concreto por eso la nueva prop en FormModel No puede usarse lo que hay porque al darle a guardar tiene que hacer una comprobación previa(beforeSave), y según eso ejecutar una cosa después del guardado onDataSaved. Me parece complicado y fácil de entender, el separar un flujo en dos funciones
jsegarra commented 2025-03-02 21:16:55 +00:00
Author
Member

Okey, ya lo tengo. Cuando me confirmes hago e2e

Okey, ya lo tengo. Cuando me confirmes hago e2e
jgallego requested review from alexm 2025-03-01 07:19:25 +00:00
jsegarra added 1 commit 2025-03-01 11:08:10 +00:00
jsegarra commented 2025-03-01 11:24:33 +00:00
Author
Member

Habian conflictos, asi que paso la tarea a WIP, hasta que valide el funcionamiento

Habian conflictos, asi que paso la tarea a WIP, hasta que valide el funcionamiento
jsegarra changed title from #6943 Customer FiscalData already exists to WIP: #6943 Customer FiscalData already exists 2025-03-01 11:24:37 +00:00
jsegarra added 2 commits 2025-03-01 19:07:46 +00:00
jsegarra added 1 commit 2025-03-01 19:12:43 +00:00
jsegarra requested review from jgallego 2025-03-02 21:16:35 +00:00
jsegarra changed title from WIP: #6943 Customer FiscalData already exists to #6943 Customer FiscalData already exists 2025-03-02 21:16:38 +00:00
jgallego approved these changes 2025-03-03 06:16:35 +00:00
src/pages/Customer/Card/CustomerFiscalData.vue
@ -34,0 +61,4 @@
openConfirmationModal(
t('Found a client with this data'),
`${t('foundClient_before')} <a class="link" href="#!/client/${exist.id}/summary" target="_blank">${exist.id}</a> ${t('foundClient_after')}`,
() => confirm(body),
jgallego commented 2025-03-03 06:16:28 +00:00
Owner

aqui llamas con data primero y luego con body, imagino que lo habrás probado, de donde sale body?

aqui llamas con _data_ primero y luego con _body_, imagino que lo habrás probado, de donde sale body?
alexm reviewed 2025-03-03 06:58:59 +00:00
src/components/FormModel.vue
@ -232,3 +236,3 @@
if ($props.saveFn) response = await $props.saveFn(body);
else response = await axios[method](url, body);
if ($props.hasConfirmModal) return;
alexm commented 2025-03-03 06:58:59 +00:00
Member

El await en saveFn al abrir el popup no se espera a que confirmes para continuar?

El await en saveFn al abrir el popup no se espera a que confirmes para continuar?
jsegarra commented 2025-03-03 10:40:54 +00:00
Author
Member

Si te refieres a la linea 236, la función saveFn apunta a formCustomFn y ahi es donde se gestiona la petición, por tanto termina la implicación de FormModel

Si te refieres a la linea 236, la función saveFn apunta a formCustomFn y ahi es donde se gestiona la petición, por tanto termina la implicación de FormModel
alexm reviewed 2025-03-03 07:00:04 +00:00
src/pages/Customer/Card/CustomerFiscalData.vue
@ -112,6 +153,7 @@ async function acceptPropagate({ isEqualizated }) {
v-model="data.sageTaxTypeFk"
data-cy="sageTaxTypeFk"
:required="data.isTaxDataChecked"
@update:model-value="hasChangedTaxData = true"
alexm commented 2025-03-03 07:00:04 +00:00
Member

Duda, esto lo han pedido?

Duda, esto lo han pedido?
jsegarra commented 2025-03-03 09:26:49 +00:00
Author
Member

Esto estaba en salix, y no se migró,

Esto estaba en salix, y no se migró,
alexm approved these changes 2025-03-03 07:00:15 +00:00
jsegarra added 1 commit 2025-03-14 11:27:55 +00:00
jsegarra closed this pull request 2025-03-14 11:28:14 +00:00
Some checks are pending
gitea/salix-front/pipeline/pr-dev Build started...
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
No reviewers
jgallego
alexm
No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
No Milestone
No Assignees
3 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#1365
No description provided.
Delete Branch "6943_customerFD_alreadyExists"

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?