verdnatura/salix
verdnatura
/
salix
3
5
Fork 2
Code Issues Pull Requests 70 Releases Wiki Activity

fix: refs #8268 refs#8268 itemShelvingAdd #3509

Open
sergiodt wants to merge 1 commits from 8268_itemShelvingAdd into master
pull from: 8268_itemShelvingAdd
merge into: verdnatura:master
Conversation 1 Commits 1 Files Changed 1 +5 -4
sergiodt commented 2025-02-28 10:10:17 +00:00
Member
Copy Link
No description provided.
sergiodt added 1 commit 2025-02-28 10:10:18 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
da21c58c24 fix: refs #8268 refs#8268 itemShelvingAdd
sergiodt requested review from jgallego 2025-02-28 10:10:34 +00:00
sergiodt commented 2025-02-28 10:12:26 +00:00
Author
Member
Copy Link

Este es un fallo que va comentar ahir Jonathan, si ho veus ok . M'ha assignat Guillermo la tarea de refactor però aixina solucionem el fallo de moment

Este es un fallo que va comentar ahir Jonathan, si ho veus ok . M'ha assignat Guillermo la tarea de refactor però aixina solucionem el fallo de moment
jgallego requested changes 2025-02-28 11:39:22 +00:00
db/routines/vn/procedures/itemShelving_add.sql
@ -57,2 +57,3 @@
AND itemFk = vItemFk
AND packing = vPacking;
AND packing = vPacking
LIMIT 1;
jgallego commented 2025-02-28 11:39:20 +00:00
Owner
Copy Link

perque un limit , quin es el problema si hi ha varios registres?
perque si fos un problema la solucio es una clau unique

perque un limit , quin es el problema si hi ha varios registres? perque si fos un problema la solucio es una clau unique
All checks were successful
gitea/salix/pipeline/pr-master This commit looks good
Required
Details
This pull request doesn't have enough approvals yet. 0 of 1 approvals granted.
You are not authorized to merge this pull request.
Sign in to join this conversation.
Reviewers
No reviewers
jgallego
Labels
Clear labels   
CR / Tests passed

Unit tests and e2e passed

  
e2e

Implement e2e

  
Missing tests

Missing tests

  
Needs refactor

   
Simple

Simple check / No CR needed

No Label
CR / Tests passed
e2e
Missing tests
Needs refactor
Simple
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#3509
No description provided.
Delete Branch "8268_itemShelvingAdd"

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?