SemianiakaVY/elsa-core
1
0
Fork 0
Code Issues Pull requests Actions 1 Packages Projects Releases Wiki Activity

Refactor

Browse source
This commit is contained in:
Gürkan Güran 2023-01-15 11:04:04 +01:00
parent 1ed7c592e9
commit f6f4a49b45
1 changed files with 4 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

7
src/designer/elsa-workflows-designer/src/modules/workflow-definitions/components/editor.tsx
View file

@ -193,10 +193,8 @@ export class WorkflowDefinitionEditor {
private saveChanges = async (): Promise<void> => {
const updatedWorkflowDefinition = this.workflowDefinitionState;
// This function is called everytime graphUpdated event is published which does not necessarily mean workflow is updated.
// To prevent redundant post requests to server, save changes only if there is a difference
// between existing workflow definition on server side and updated workflow definition on client side.
if (await this.hasWorkflowDefinitionAnyUpdatedData(updatedWorkflowDefinition)) {
// If workflow definition is published, override the latest version.
if (updatedWorkflowDefinition.isPublished) {
updatedWorkflowDefinition.version = this.workflowVersions.find(v => v.isLatest).version;
}
@ -204,6 +202,9 @@ export class WorkflowDefinitionEditor {
}
};
// This function is called everytime graphUpdated event is published which does not necessarily mean workflow is updated.
// To prevent redundant post requests to server, save changes only if there is a difference
// between existing workflow definition on server side and updated workflow definition on client side.
private hasWorkflowDefinitionAnyUpdatedData = async (updatedWorkflowDefinition : WorkflowDefinition): Promise<boolean> => {
const existingWorkflowDefinition = await this.workflowDefinitionApi.get({definitionId: updatedWorkflowDefinition.definitionId, versionOptions: {version: updatedWorkflowDefinition.version}});