verdnatura/salix
verdnatura
/
salix
3
5
Fork 0
Code Issues Pull Requests 64 Releases Wiki Activity

refs #6276 - notifyNewWarehouse #2186

Closed
sergiodt wants to merge 8 commits from 6276-nodifyNewWarehouse into master
pull from: 6276-nodifyNewWarehouse
merge into: verdnatura:master
Conversation 0 Commits 8 Files Changed 10 +25 -26
sergiodt commented 2024-03-20 15:15:06 +00:00
Member
Copy Link
No description provided.
sergiodt added 2 commits 2024-03-20 15:15:06 +00:00
gitea/salix/pipeline/pr-test Build queued... Details
7a06d1217e fix: refs #6276 test & procedure
sergiodt requested review from alexm 2024-03-20 15:15:13 +00:00
sergiodt added 1 commit 2024-03-20 15:15:20 +00:00
gitea/salix/pipeline/pr-test This commit looks good Details
5e903e9be3 Merge branch 'test' into 6276-nodifyNewWarehouse
alexm changed title from 6276-nodifyNewWarehouse to refs #6276 - notifyNewWarehouse 2024-03-20 15:47:16 +00:00
alexm requested changes 2024-03-20 15:51:26 +00:00
modules/ticket/back/methods/sale-tracking/setPicked.js Outdated
@ -89,16 +89,9 @@ module.exports = Self => {
const {itemOriginalFk} = await models.Buy.findById(buyFk, {fields: ['itemOriginalFk']}, myOptions);
if (itemOriginalFk) await models.SaleBuy.create({saleFk, buyFk}, myOptions);
} catch (e) {
alexm commented 2024-03-20 15:51:12 +00:00
Member
Copy Link

Sustituir catch(e) por finally

Sustituir `catch(e)` por `finally`
sergiodt marked this conversation as resolved
sergiodt added 1 commit 2024-03-21 06:13:25 +00:00
gitea/salix/pipeline/pr-test This commit looks good Details
gitea/salix/pipeline/pr-master This commit looks good Details
6f0cfd421e refs #6276 feat:newWarehouse
sergiodt changed target branch from test to master 2024-03-21 08:17:27 +00:00
sergiodt added 1 commit 2024-03-21 08:32:12 +00:00
gitea/salix/pipeline/pr-master There was a failure building this commit Details
025e53634a refs #6276 feat:newWarehouse
sergiodt added 1 commit 2024-03-21 08:32:47 +00:00
gitea/salix/pipeline/pr-master There was a failure building this commit Details
d7b70bea4b Merge branch 'master' into 6276-nodifyNewWarehouse
sergiodt added 2 commits 2024-03-21 08:33:33 +00:00
sergiodt closed this pull request 2024-03-25 16:48:46 +00:00
Some checks are pending
gitea/salix/pipeline/pr-master Build queued...
Required
Details

Pull request closed

Please reopen this pull request to perform a merge.
Sign in to join this conversation.
Reviewers
No reviewers
alexm
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#2186
No description provided.
Delete Branch "6276-nodifyNewWarehouse"

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?