From d84192d1e6cc9ba8b5b944b3c01ac206014c0d52 Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Tue, 22 Nov 2022 11:35:45 +0200
Subject: [PATCH 1/2] feat: Add share button to workflows list

---
 .../src/components/MainHeader/WorkflowDetails.vue    |  2 +-
 packages/editor-ui/src/components/Modals.vue         |  8 +++++++-
 packages/editor-ui/src/components/WorkflowCard.vue   |  9 ++++++++-
 .../src/components/WorkflowShareModal.ee.vue         | 12 +++++++++---
 packages/editor-ui/src/plugins/i18n/locales/en.json  |  1 +
 5 files changed, 26 insertions(+), 6 deletions(-)

diff --git a/packages/editor-ui/src/components/MainHeader/WorkflowDetails.vue b/packages/editor-ui/src/components/MainHeader/WorkflowDetails.vue
index 5e43bbe1a4f8b..0cff15cddce53 100644
--- a/packages/editor-ui/src/components/MainHeader/WorkflowDetails.vue
+++ b/packages/editor-ui/src/components/MainHeader/WorkflowDetails.vue
@@ -241,7 +241,7 @@ export default mixins(workflowHelpers, titleChange).extend({
 			if (saved) await this.settingsStore.fetchPromptsData();
 		},
 		onShareButtonClick() {
-			this.uiStore.openModal(WORKFLOW_SHARE_MODAL_KEY);
+			this.uiStore.openModalWithData({ name: WORKFLOW_SHARE_MODAL_KEY, data: { id: this.currentWorkflowId } });
 		},
 		onTagsEditEnable() {
 			this.$data.appliedTagIds = this.currentWorkflowTagIds;
diff --git a/packages/editor-ui/src/components/Modals.vue b/packages/editor-ui/src/components/Modals.vue
index aa66a3bddb9f5..ce072545d3d33 100644
--- a/packages/editor-ui/src/components/Modals.vue
+++ b/packages/editor-ui/src/components/Modals.vue
@@ -83,7 +83,13 @@
 		</ModalRoot>
 
 		<ModalRoot :name="WORKFLOW_SHARE_MODAL_KEY">
-			<WorkflowShareModal />
+			<template #default="{ modalName, active, data }">
+				<WorkflowShareModal
+					:data="data"
+					:isActive="active"
+					:modalName="modalName"
+				/>
+			</template>
 		</ModalRoot>
 
 		<ModalRoot :name="ONBOARDING_CALL_SIGNUP_MODAL_KEY">
diff --git a/packages/editor-ui/src/components/WorkflowCard.vue b/packages/editor-ui/src/components/WorkflowCard.vue
index 8b1e185efa764..466f52cd11fbe 100644
--- a/packages/editor-ui/src/components/WorkflowCard.vue
+++ b/packages/editor-ui/src/components/WorkflowCard.vue
@@ -59,7 +59,7 @@
 <script lang="ts">
 import mixins from 'vue-typed-mixins';
 import {IWorkflowDb, IUser, ITag} from "@/Interface";
-import {DUPLICATE_MODAL_KEY, EnterpriseEditionFeature, VIEWS} from '@/constants';
+import {DUPLICATE_MODAL_KEY, EnterpriseEditionFeature, VIEWS, WORKFLOW_SHARE_MODAL_KEY} from '@/constants';
 import {showMessage} from "@/components/mixins/showMessage";
 import {getWorkflowPermissions, IPermissions} from "@/permissions";
 import dateformat from "dateformat";
@@ -74,6 +74,7 @@ import { useWorkflowsStore } from '@/stores/workflows';
 
 export const WORKFLOW_LIST_ITEM_ACTIONS = {
 	OPEN: 'open',
+	SHARE: 'share',
 	DUPLICATE: 'duplicate',
 	DELETE: 'delete',
 };
@@ -131,6 +132,10 @@ export default mixins(
 					label: this.$locale.baseText('workflows.item.open'),
 					value: WORKFLOW_LIST_ITEM_ACTIONS.OPEN,
 				},
+				{
+					label: this.$locale.baseText('workflows.item.share'),
+					value: WORKFLOW_LIST_ITEM_ACTIONS.SHARE,
+				},
 				{
 					label: this.$locale.baseText('workflows.item.duplicate'),
 					value: WORKFLOW_LIST_ITEM_ACTIONS.DUPLICATE,
@@ -183,6 +188,8 @@ export default mixins(
 						tags: (this.data.tags || []).map((tag: ITag) => tag.id),
 					},
 				});
+			} else if (action === WORKFLOW_LIST_ITEM_ACTIONS.SHARE) {
+				this.uiStore.openModalWithData({ name: WORKFLOW_SHARE_MODAL_KEY, data: { id: this.data.id } });
 			} else if (action === WORKFLOW_LIST_ITEM_ACTIONS.DELETE) {
 				const deleteConfirmed = await this.confirmMessage(
 					this.$locale.baseText(
diff --git a/packages/editor-ui/src/components/WorkflowShareModal.ee.vue b/packages/editor-ui/src/components/WorkflowShareModal.ee.vue
index 3fd05ee7fe3f4..85a3d24272079 100644
--- a/packages/editor-ui/src/components/WorkflowShareModal.ee.vue
+++ b/packages/editor-ui/src/components/WorkflowShareModal.ee.vue
@@ -120,6 +120,12 @@ export default mixins(
 	components: {
 		Modal,
 	},
+	props: {
+		data: {
+			type: Object,
+			default: () => ({}),
+		},
+	},
 	data() {
 		const workflowsStore = useWorkflowsStore();
 
@@ -127,7 +133,7 @@ export default mixins(
 			WORKFLOW_SHARE_MODAL_KEY,
 			loading: false,
 			modalBus: new Vue(),
-			sharedWith: [...(workflowsStore.workflow.sharedWith || [])] as Array<Partial<IUser>>,
+			sharedWith: [...(workflowsStore.workflowsById[this.data.id].sharedWith || [])] as Array<Partial<IUser>>,
 			EnterpriseEditionFeature,
 		};
 	},
@@ -150,7 +156,7 @@ export default mixins(
 			] as Array<Partial<IUser>>).concat(this.sharedWith || []);
 		},
 		workflow(): IWorkflowDb {
-			return this.workflowsStore.workflow;
+			return this.workflowsStore.workflowsById[this.data.id];
 		},
 		currentUser(): IUser | null {
 			return this.usersStore.currentUser;
@@ -221,7 +227,7 @@ export default mixins(
 			}
 
 			if (confirm) {
-				this.sharedWith = this.sharedWith.filter((sharee: IUser) => {
+				this.sharedWith = this.sharedWith.filter((sharee: Partial<IUser>) => {
 					return sharee.id !== user.id;
 				});
 			}
diff --git a/packages/editor-ui/src/plugins/i18n/locales/en.json b/packages/editor-ui/src/plugins/i18n/locales/en.json
index 8900502d9c72f..b3781d9298cf2 100644
--- a/packages/editor-ui/src/plugins/i18n/locales/en.json
+++ b/packages/editor-ui/src/plugins/i18n/locales/en.json
@@ -1342,6 +1342,7 @@
 	"workflows.menu.my": "My workflows",
 	"workflows.menu.all": "All workflows",
 	"workflows.item.open": "Open",
+	"workflows.item.share": "Share...",
 	"workflows.item.duplicate": "Duplicate",
 	"workflows.item.delete": "Delete",
 	"workflows.item.updated": "Last updated",

From 353ee115d6294cfe723bccb2f535536fd47cd989 Mon Sep 17 00:00:00 2001
From: Alex Grozav <alex@grozav.com>
Date: Tue, 22 Nov 2022 11:38:47 +0200
Subject: [PATCH 2/2] fix: take empty workflow into account when sharing

---
 .../editor-ui/src/components/WorkflowShareModal.ee.vue |  9 +++++++--
 .../editor-ui/src/components/mixins/workflowHelpers.ts | 10 +++++++++-
 2 files changed, 16 insertions(+), 3 deletions(-)

diff --git a/packages/editor-ui/src/components/WorkflowShareModal.ee.vue b/packages/editor-ui/src/components/WorkflowShareModal.ee.vue
index 85a3d24272079..d5e6e8ab57004 100644
--- a/packages/editor-ui/src/components/WorkflowShareModal.ee.vue
+++ b/packages/editor-ui/src/components/WorkflowShareModal.ee.vue
@@ -128,12 +128,15 @@ export default mixins(
 	},
 	data() {
 		const workflowsStore = useWorkflowsStore();
+		const workflow = this.data.id === PLACEHOLDER_EMPTY_WORKFLOW_ID
+			? workflowsStore.workflow
+			: workflowsStore.workflowsById[this.data.id];
 
 		return {
 			WORKFLOW_SHARE_MODAL_KEY,
 			loading: false,
 			modalBus: new Vue(),
-			sharedWith: [...(workflowsStore.workflowsById[this.data.id].sharedWith || [])] as Array<Partial<IUser>>,
+			sharedWith: [...(workflow.sharedWith || [])] as Array<Partial<IUser>>,
 			EnterpriseEditionFeature,
 		};
 	},
@@ -156,7 +159,9 @@ export default mixins(
 			] as Array<Partial<IUser>>).concat(this.sharedWith || []);
 		},
 		workflow(): IWorkflowDb {
-			return this.workflowsStore.workflowsById[this.data.id];
+			return this.data.id === PLACEHOLDER_EMPTY_WORKFLOW_ID
+				? this.workflowsStore.workflow
+				: this.workflowsStore.workflowsById[this.data.id];
 		},
 		currentUser(): IUser | null {
 			return this.usersStore.currentUser;
diff --git a/packages/editor-ui/src/components/mixins/workflowHelpers.ts b/packages/editor-ui/src/components/mixins/workflowHelpers.ts
index 8ae4973da2395..24d1b9aeb9e0c 100644
--- a/packages/editor-ui/src/components/mixins/workflowHelpers.ts
+++ b/packages/editor-ui/src/components/mixins/workflowHelpers.ts
@@ -4,7 +4,7 @@ import {
 	PLACEHOLDER_EMPTY_WORKFLOW_ID,
 	START_NODE_TYPE,
 	WEBHOOK_NODE_TYPE,
-	VIEWS,
+	VIEWS, EnterpriseEditionFeature,
 } from '@/constants';
 
 import {
@@ -65,6 +65,8 @@ import { IWorkflowSettings } from 'n8n-workflow';
 import { useNDVStore } from '@/stores/ndv';
 import { useTemplatesStore } from '@/stores/templates';
 import { useNodeTypesStore } from '@/stores/nodeTypes';
+import useWorkflowsEEStore from "@/stores/workflows.ee";
+import {useUsersStore} from "@/stores/users";
 
 let cachedWorkflowKey: string | null = '';
 let cachedWorkflow: Workflow | null = null;
@@ -83,6 +85,8 @@ export const workflowHelpers = mixins(
 				useRootStore,
 				useTemplatesStore,
 				useWorkflowsStore,
+				useWorkflowsEEStore,
+				useUsersStore,
 				useUIStore,
 			),
 		},
@@ -796,6 +800,10 @@ export const workflowHelpers = mixins(
 					this.workflowsStore.addWorkflow(workflowData);
 					this.workflowsStore.setWorkflowHash(workflowData.hash);
 
+					if (this.settingsStore.isEnterpriseFeatureEnabled(EnterpriseEditionFeature.WorkflowSharing) && this.usersStore.currentUser) {
+						this.workflowsEEStore.setWorkflowOwnedBy({ workflowId: workflowData.id, ownedBy: this.usersStore.currentUser });
+					}
+
 					if (openInNewWindow) {
 						const routeData = this.$router.resolve({name: VIEWS.WORKFLOW, params: {name: workflowData.id}});
 						window.open(routeData.href, '_blank');