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

HOTFIX: ticketSale keyup.enter #1009

Merged
jsegarra merged 5 commits from hotfix_ticketSale_enter into master 2024-11-28 10:03:40 +00:00
Conversation 4 Commits 5 Files Changed 1 +25 -25
jsegarra commented 2024-11-27 00:16:23 +00:00
Member
Copy Link
No description provided.
jsegarra added 1 commit 2024-11-27 00:16:24 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e3bdea2197 fix: ticketSale keyup.enter
jsegarra requested review from jgallego 2024-11-27 00:16:31 +00:00
jsegarra requested review from carlossa 2024-11-27 00:16:31 +00:00
jgallego commented 2024-11-27 06:25:59 +00:00
Owner
Copy Link

@jsegarra aqui necesito que me expliques problema y como lo has solucionado

@jsegarra aqui necesito que me expliques problema y como lo has solucionado
jsegarra commented 2024-11-27 06:35:11 +00:00
Author
Member
Copy Link

@jsegarra aqui necesito que me expliques problema y como lo has solucionado

El motivo de este cambio es que en Salix al cambiar la cantidad y pulsar el entre se actualizaba.
Ahora tenía que hacer click fuera y me lo han reportado.
La solución, ha sido que al modificar el valor, la línea se añade en una variable, al hacer blur o enter se actualiza y se vacía la variable así si se dispara n eventos a posterior del primero no se ejecuta porque tiene un return. La técnica de la bandera

> @jsegarra aqui necesito que me expliques problema y como lo has solucionado El motivo de este cambio es que en Salix al cambiar la cantidad y pulsar el entre se actualizaba. Ahora tenía que hacer click fuera y me lo han reportado. La solución, ha sido que al modificar el valor, la línea se añade en una variable, al hacer blur o enter se actualiza y se vacía la variable así si se dispara n eventos a posterior del primero no se ejecuta porque tiene un return. La técnica de la bandera
jsegarra added 1 commit 2024-11-27 07:40:27 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
dc156e44b6 Merge branch 'master' into hotfix_ticketSale_enter
jsegarra requested review from juan 2024-11-27 08:33:24 +00:00
jsegarra added 1 commit 2024-11-27 18:20:02 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e23c6bc144 Merge branch 'master' into hotfix_ticketSale_enter
jgallego approved these changes 2024-11-28 07:05:38 +00:00
jsegarra added 1 commit 2024-11-28 07:35:48 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
8daa08b99c Merge branch 'master' into hotfix_ticketSale_enter
jsegarra commented 2024-11-28 08:39:10 +00:00
Author
Member
Copy Link

@jgallego Lo comenté ayer con @juan, lo ve bien si corrige un fallo y en breve se va a reemplazar por lo que está implementando Pablo.
Lo comento por si, podemos eliminar a satorres y/o juan de las aprobaciones para que lo tengan disponible

@jgallego Lo comenté ayer con @juan, lo ve bien si corrige un fallo y en breve se va a reemplazar por lo que está implementando Pablo. Lo comento por si, podemos eliminar a satorres y/o juan de las aprobaciones para que lo tengan disponible
jsegarra added 1 commit 2024-11-28 08:39:20 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
8bc3c1610b Merge branch 'master' into hotfix_ticketSale_enter
jgallego removed review request for carlossa 2024-11-28 09:52:11 +00:00
jgallego removed review request for juan 2024-11-28 09:52:12 +00:00
jgallego commented 2024-11-28 09:53:03 +00:00
Owner
Copy Link

@jgallego Lo comenté ayer con @juan, lo ve bien si corrige un fallo y en breve se va a reemplazar por lo que está implementando Pablo.
Lo comento por si, podemos eliminar a satorres y/o juan de las aprobaciones para que lo tengan disponible

@jsegarra hecho

> @jgallego Lo comenté ayer con @juan, lo ve bien si corrige un fallo y en breve se va a reemplazar por lo que está implementando Pablo. > Lo comento por si, podemos eliminar a satorres y/o juan de las aprobaciones para que lo tengan disponible @jsegarra hecho
jsegarra merged commit 1bbce5f828 into master 2024-11-28 10:03:40 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
Labels
Clear labels   
bug

Something is not working

  
CR / Tests passed

CR / Tests passed

  
duplicate

This issue or pull request already exists

  
enhancement

New feature

  
invalid

Something is wrong

  
Missing Tests

Missing Tests

  
wontfix

This won't be fixed

No Label
bug
CR / Tests passed
duplicate
enhancement
invalid
Missing Tests
wontfix
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
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#1009
No description provided.
Delete Branch "hotfix_ticketSale_enter"

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?