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

feat: implement onBeforeSave function to handle form data updates #1629

Merged
alexm merged 2 commits from hotFix_claim_basicData into master 2025-03-24 14:10:28 +00:00
Conversation 1 Commits 2 Files Changed 1 +9 -1
alexm commented 2025-03-24 13:02:00 +00:00
Member
Copy Link
No description provided.
alexm added 1 commit 2025-03-24 13:02:00 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
e1ef6f87f3 feat: implement onBeforeSave function to handle form data updates
alexm requested review from jsegarra 2025-03-24 13:02:14 +00:00
jsegarra requested changes 2025-03-24 13:09:59 +00:00
Dismissed
jsegarra left a comment
Member
Copy Link

La idea es quitar todos los mapper que sean repetitivos.
Con tu propuesta hacemos lo contrario
Creo que habia un problema en FormModel( o algo se ha cambiado) yestaba definidos como null y el operador ternario no se ejecutaba
con el zip que te he enviado ya tiene valor por defecto

La idea es quitar todos los mapper que sean repetitivos. Con tu propuesta hacemos lo contrario Creo que habia un problema en FormModel( o algo se ha cambiado) yestaba definidos como null y el operador ternario no se ejecutaba con el zip que te he enviado ya tiene valor por defecto
alexm commented 2025-03-24 13:19:48 +00:00
Author
Member
Copy Link

La idea es quitar todos los mapper que sean repetitivos.
Con tu propuesta hacemos lo contrario
Creo que habia un problema en FormModel( o algo se ha cambiado) yestaba definidos como null y el operador ternario no se ejecutaba
con el zip que te he enviado ya tiene valor por defecto

Ese cambio no lo veo para producción directo sin probar ni nada.
Ademas tiene codigo que no se usa:
image

Se deberia de haber hecho en la tarea correspondiente https://redmine.verdnatura.es/issues/8217
@jsegarra

> La idea es quitar todos los mapper que sean repetitivos. > Con tu propuesta hacemos lo contrario > Creo que habia un problema en FormModel( o algo se ha cambiado) yestaba definidos como null y el operador ternario no se ejecutaba > con el zip que te he enviado ya tiene valor por defecto Ese cambio no lo veo para producción directo sin probar ni nada. Ademas tiene codigo que no se usa: ![image](/attachments/1f0f2ce0-6c35-45d3-9378-aa1249f9bc9a) Se deberia de haber hecho en la tarea correspondiente https://redmine.verdnatura.es/issues/8217 @jsegarra
image.png
38 KiB
alexm requested review from jsegarra 2025-03-24 13:21:57 +00:00
alexm added 1 commit 2025-03-24 13:52:55 +00:00
gitea/salix-front/pipeline/pr-master This commit looks good Details
ec456b5ac8 Merge branch 'master' into hotFix_claim_basicData
jsegarra approved these changes 2025-03-24 13:58:48 +00:00
alexm merged commit a2594d2ab0 into master 2025-03-24 14:10:28 +00:00
alexm deleted branch hotFix_claim_basicData 2025-03-24 14:10:28 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
jsegarra
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#1629
No description provided.
Delete Branch "hotFix_claim_basicData"

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?