diff --git a/packages/cli/src/workflows/workflows.controller.ee.ts b/packages/cli/src/workflows/workflows.controller.ee.ts index b24d1b7c11e7e..292e1092ec082 100644 --- a/packages/cli/src/workflows/workflows.controller.ee.ts +++ b/packages/cli/src/workflows/workflows.controller.ee.ts @@ -211,7 +211,7 @@ EEWorkflowController.patch( '/:id(\\d+)', ResponseHelper.send(async (req: WorkflowRequest.Update) => { const { id: workflowId } = req.params; - const forceSave = req.query.forceSave === 'true'; + // const forceSave = req.query.forceSave === 'true'; // disabled temporarily - tests were also disabled const updateData = new WorkflowEntity(); const { tags, ...rest } = req.body; @@ -224,7 +224,7 @@ EEWorkflowController.patch( safeWorkflow, workflowId, tags, - forceSave, + true, ); const { id, ...remainder } = updatedWorkflow; diff --git a/packages/cli/src/workflows/workflows.controller.ts b/packages/cli/src/workflows/workflows.controller.ts index ae4831ac7b181..0ad5bcd0ddc06 100644 --- a/packages/cli/src/workflows/workflows.controller.ts +++ b/packages/cli/src/workflows/workflows.controller.ts @@ -248,7 +248,7 @@ workflowsController.patch( updateData, workflowId, tags, - false, + true, ['owner'], ); diff --git a/packages/cli/test/integration/workflows.controller.ee.test.ts b/packages/cli/test/integration/workflows.controller.ee.test.ts index 8a9a249d95ad3..4363f25f1443d 100644 --- a/packages/cli/test/integration/workflows.controller.ee.test.ts +++ b/packages/cli/test/integration/workflows.controller.ee.test.ts @@ -731,7 +731,7 @@ describe('PATCH /workflows/:id - validate credential permissions to user', () => }); describe('PATCH /workflows/:id - validate interim updates', () => { - it('should block owner updating workflow nodes on interim update by member', async () => { + xit('should block owner updating workflow nodes on interim update by member', async () => { const owner = await testDb.createUser({ globalRole: globalOwnerRole }); const member = await testDb.createUser({ globalRole: globalMemberRole }); @@ -764,7 +764,7 @@ describe('PATCH /workflows/:id - validate interim updates', () => { ); }); - it('should block member updating workflow nodes on interim update by owner', async () => { + xit('should block member updating workflow nodes on interim update by owner', async () => { const owner = await testDb.createUser({ globalRole: globalOwnerRole }); const member = await testDb.createUser({ globalRole: globalMemberRole }); @@ -806,7 +806,7 @@ describe('PATCH /workflows/:id - validate interim updates', () => { ); }); - it('should block owner activation on interim activation by member', async () => { + xit('should block owner activation on interim activation by member', async () => { const owner = await testDb.createUser({ globalRole: globalOwnerRole }); const member = await testDb.createUser({ globalRole: globalMemberRole }); @@ -836,7 +836,7 @@ describe('PATCH /workflows/:id - validate interim updates', () => { ); }); - it('should block member activation on interim activation by owner', async () => { + xit('should block member activation on interim activation by owner', async () => { const owner = await testDb.createUser({ globalRole: globalOwnerRole }); const member = await testDb.createUser({ globalRole: globalMemberRole }); @@ -875,7 +875,7 @@ describe('PATCH /workflows/:id - validate interim updates', () => { ); }); - it('should block member updating workflow settings on interim update by owner', async () => { + xit('should block member updating workflow settings on interim update by owner', async () => { const owner = await testDb.createUser({ globalRole: globalOwnerRole }); const member = await testDb.createUser({ globalRole: globalMemberRole }); @@ -910,7 +910,7 @@ describe('PATCH /workflows/:id - validate interim updates', () => { ); }); - it('should block member updating workflow name on interim update by owner', async () => { + xit('should block member updating workflow name on interim update by owner', async () => { const owner = await testDb.createUser({ globalRole: globalOwnerRole }); const member = await testDb.createUser({ globalRole: globalMemberRole });