verdnatura/salix
verdnatura
/
salix
3
4
Fork 0
Code Issues Pull Requests 57 Releases Wiki Activity

fix: refs #6861 saleTrackingAdd #3121

Merged
sergiodt merged 4 commits from 6861-ReserveSaleTracking into master 2024-10-17 15:29:13 +00:00
Conversation 0 Commits 4 Files Changed 1 +20
sergiodt commented 2024-10-17 12:01:50 +00:00
Member
Copy Link

He deixat itemShelvingSale isPicked ja que hi ha un trigger en sale i no deixa posar-ho. No crec que estiga mal posant itemShelvingSale perquè en el pitjor dels casos que aparega algo que has preparat

He deixat itemShelvingSale isPicked ja que hi ha un trigger en sale i no deixa posar-ho. No crec que estiga mal posant itemShelvingSale perquè en el pitjor dels casos que aparega algo que has preparat
sergiodt added 1 commit 2024-10-17 12:01:50 +00:00
gitea/salix/pipeline/pr-master There was a failure building this commit Details
cb53962405 fix: refs #6861 saleTrackingAdd
sergiodt requested review from carlosap 2024-10-17 12:01:59 +00:00
carlosap requested changes 2024-10-17 14:29:54 +00:00
Dismissed
db/routines/vn/procedures/saleTracking_sectorCollectionAddPrevOK.sql Outdated
@ -0,0 +1,26 @@
DELIMITER $$
CREATE OR REPLACE DEFINER=`vn`@`localhost` PROCEDURE `vn`.`saleTracking_sectorCollectionAddPrevOK`(vSectorCollectionFk INT)
carlosap commented 2024-10-17 14:25:54 +00:00
Member
Copy Link

Salto de línea

Salto de línea
sergiodt marked this conversation as resolved
db/routines/vn/procedures/saleTracking_sectorCollectionAddPrevOK.sql Outdated
@ -0,0 +13,4 @@
stateFk
)
SELECT sgd.saleFk,
TRUE,
carlosap commented 2024-10-17 14:26:59 +00:00
Member
Copy Link

Tabulación

Tabulación
sergiodt marked this conversation as resolved
db/routines/vn/procedures/saleTracking_sectorCollectionAddPrevOK.sql Outdated
@ -0,0 +16,4 @@
TRUE,
sc.userFk,
s.id
FROM vn.sectorCollection sc
carlosap commented 2024-10-17 14:27:20 +00:00
Member
Copy Link

esquema no es necesario

esquema no es necesario
sergiodt marked this conversation as resolved
sergiodt added 1 commit 2024-10-17 15:01:56 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
63c0ec308f fix: refs #6861 saleTrackingAdd
sergiodt requested review from carlosap 2024-10-17 15:03:05 +00:00
carlosap requested changes 2024-10-17 15:08:33 +00:00
Dismissed
db/routines/vn/procedures/saleTracking_sectorCollectionAddPrevOK.sql Outdated
@ -0,0 +1,28 @@
DELIMITER $$
CREATE OR REPLACE DEFINER=`vn`@`localhost` PROCEDURE `vn`.`saleTracking_sectorCollectionAddPrevOK`(
vSectorCollectionFk INT
)
carlosap commented 2024-10-17 15:07:06 +00:00
Member
Copy Link

Tabulación ) -1

Tabulación ) -1
sergiodt marked this conversation as resolved
db/routines/vn/procedures/saleTracking_sectorCollectionAddPrevOK.sql Outdated
@ -0,0 +9,4 @@
* @param vSectorCollectionFk Identificador de vn.sectorCollection
*/
REPLACE saleTracking(
saleFk,
carlosap commented 2024-10-17 15:07:32 +00:00
Member
Copy Link

Tabulación

Tabulación
sergiodt marked this conversation as resolved
db/routines/vn/procedures/saleTracking_sectorCollectionAddPrevOK.sql Outdated
@ -0,0 +12,4 @@
saleFk,
isChecked,
workerFk,
stateFk
carlosap commented 2024-10-17 15:08:07 +00:00
Member
Copy Link

Tabulación )

Tabulación )
sergiodt marked this conversation as resolved
sergiodt added 1 commit 2024-10-17 15:20:33 +00:00
gitea/salix/pipeline/pr-master There was a failure building this commit Details
3463036086 fix: refs #6861 saleTrackingAdd
sergiodt requested review from carlosap 2024-10-17 15:20:57 +00:00
carlosap approved these changes 2024-10-17 15:21:57 +00:00
sergiodt added 1 commit 2024-10-17 15:23:08 +00:00
gitea/salix/pipeline/pr-master This commit looks good Details
8df09bc64a Merge branch 'master' into 6861-ReserveSaleTracking
sergiodt merged commit 90a3f369b6 into master 2024-10-17 15:29:13 +00:00
sergiodt deleted branch 6861-ReserveSaleTracking 2024-10-17 15:29:13 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
carlosap
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#3121
No description provided.
Delete Branch "6861-ReserveSaleTracking"

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?