fix: refs #7133 handleSalesModelValue function to handle empty input #1116

Merged
jorgep merged 3 commits from 7133-fixCustomerSelect into dev 2024-12-23 15:56:33 +00:00
Member
No description provided.
jorgep added 1 commit 2024-12-19 11:16:03 +00:00
jorgep added 1 commit 2024-12-19 11:16:15 +00:00
jorgep requested review from jgallego 2024-12-19 11:16:23 +00:00
Author
Member

@jgallego Confirmame que es esto lo que se pedía. Creo haber entendido que era comprobar esto:

  1. Si quiero cambiar el valor, no aprieto el aspa sino la flecha y elijo otro valor
    Si lo quiero eliminar, pulso el aspa y guardo el registro

He visto que funcionaba como toca.

Y arreglar esto:
Actualmente el componente tiene un fallo pasos para provocarlo: https://dev-lilium.verdnatura.es/#/customer/42190/basic-data

  1. Aprietamos el aspa para eliminar El contenido.
  2. Elegimos el primer registro.
  3. Volvemos a apretar el aspa para eliminar el contenido.
  4. Al desplegar ya no aparece ningún registro, esto es un error

Que solo pasaba en el deplegable de salesPerson

@jgallego Confirmame que es esto lo que se pedía. Creo haber entendido que era comprobar esto: 1. Si quiero cambiar el valor, no aprieto el aspa sino la flecha y elijo otro valor Si lo quiero eliminar, pulso el aspa y guardo el registro He visto que funcionaba como toca. Y arreglar esto: Actualmente el componente tiene un fallo pasos para provocarlo: https://dev-lilium.verdnatura.es/#/customer/42190/basic-data 1. Aprietamos el aspa para eliminar El contenido. 2. Elegimos el primer registro. 3. Volvemos a apretar el aspa para eliminar el contenido. 4. Al desplegar ya no aparece ningún registro, esto es un error Que solo pasaba en el deplegable de salesPerson
jorgep added the
CR / Tests passed
label 2024-12-19 11:54:04 +00:00
jgallego requested changes 2024-12-19 15:50:16 +00:00
Dismissed
@ -21,2 +15,2 @@
],
});
const handleSalesModelValue = (val) => {
if (!val) val = '';
Owner

has puesto el mismo codigo en los dos sitios, no lo podemo poner mas arriba comun a lo que sea susceptible de usarlo?

has puesto el mismo codigo en los dos sitios, no lo podemo poner mas arriba comun a lo que sea susceptible de usarlo?
Author
Member

No lo he hecho porque he visto que se usa diferente filtro para buscar en esta url Workers/search y creo que debería estar unificado para todos en el componente ya creado VnSelectWorker.

No lo he hecho porque he visto que se usa diferente filtro para buscar en esta url Workers/search y creo que debería estar unificado para todos en el componente ya creado VnSelectWorker.
Owner

si hace falta crea redmine

si hace falta crea redmine
Author
Member

Se usa la misma url en varios sitios:

image

Lo unico que cambia es el dpto. pero la fn yo la metería de forma general para ese componente

image

Se usa la misma url en varios sitios: ![image](/attachments/ce310b84-acdc-4543-8a42-54205fb032be) Lo unico que cambia es el dpto. pero la fn yo la metería de forma general para ese componente ![image](/attachments/b227d8c2-c326-4f66-bfb2-d41e29e5d1f8)
jorgep requested review from jgallego 2024-12-23 12:39:54 +00:00
jgallego approved these changes 2024-12-23 13:25:44 +00:00
jorgep added 1 commit 2024-12-23 15:55:27 +00:00
jorgep merged commit 7ef1e49277 into dev 2024-12-23 15:56:33 +00:00
jorgep deleted branch 7133-fixCustomerSelect 2024-12-23 15:56:33 +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#1116
No description provided.