From 06661745019e95423aaedeaa8a5f134880ce5cd6 Mon Sep 17 00:00:00 2001
From: "kasia.oczkowska" <katarzyna.oczkowska@theworkingroup.net>
Date: Thu, 4 Jul 2024 14:08:35 +0100
Subject: [PATCH 1/2] fix: fix error for only Create role when publishing flow

---
 .../web/src/components/EditorLayout/index.jsx | 34 ++++++-------------
 1 file changed, 11 insertions(+), 23 deletions(-)

diff --git a/packages/web/src/components/EditorLayout/index.jsx b/packages/web/src/components/EditorLayout/index.jsx
index 37819435..690f4625 100644
--- a/packages/web/src/components/EditorLayout/index.jsx
+++ b/packages/web/src/components/EditorLayout/index.jsx
@@ -36,23 +36,18 @@ export default function EditorLayout() {
 
   const onFlowNameUpdate = React.useCallback(
     async (name) => {
-      await updateFlow({
-        variables: {
-          input: {
-            id: flowId,
-            name,
+      try {
+        await updateFlow({
+          variables: {
+            input: {
+              id: flowId,
+              name,
+            },
           },
-        },
-        optimisticResponse: {
-          updateFlow: {
-            __typename: 'Flow',
-            id: flowId,
-            name,
-          },
-        },
-      });
+        });
 
-      await queryClient.invalidateQueries({ queryKey: ['flows', flowId] });
+        await queryClient.invalidateQueries({ queryKey: ['flows', flowId] });
+      } catch (e) {}
     },
     [flowId, queryClient],
   );
@@ -67,13 +62,6 @@ export default function EditorLayout() {
               active,
             },
           },
-          optimisticResponse: {
-            updateFlowStatus: {
-              __typename: 'Flow',
-              id: flowId,
-              active,
-            },
-          },
         });
 
         await queryClient.invalidateQueries({ queryKey: ['flows', flowId] });
@@ -126,7 +114,7 @@ export default function EditorLayout() {
           <Button
             variant="contained"
             size="small"
-            onClick={() => onFlowStatusUpdate(!flow.active)}
+            onClick={() => onFlowStatusUpdate(flow ? !flow.active : false)}
             data-test={
               flow?.active ? 'unpublish-flow-button' : 'publish-flow-button'
             }

From 0966f9d715369f47cebb43f02c02c13a9a42ed53 Mon Sep 17 00:00:00 2001
From: "kasia.oczkowska" <katarzyna.oczkowska@theworkingroup.net>
Date: Thu, 11 Jul 2024 09:39:31 +0100
Subject: [PATCH 2/2] feat: disable publish button when no flow or permissions

---
 .../web/src/components/EditorLayout/index.jsx | 30 +++++++++++--------
 1 file changed, 18 insertions(+), 12 deletions(-)

diff --git a/packages/web/src/components/EditorLayout/index.jsx b/packages/web/src/components/EditorLayout/index.jsx
index 690f4625..92bd6441 100644
--- a/packages/web/src/components/EditorLayout/index.jsx
+++ b/packages/web/src/components/EditorLayout/index.jsx
@@ -14,6 +14,7 @@ import { EditorProvider } from 'contexts/Editor';
 import EditableTypography from 'components/EditableTypography';
 import Container from 'components/Container';
 import Editor from 'components/Editor';
+import Can from 'components/Can';
 import useFormatMessage from 'hooks/useFormatMessage';
 import { UPDATE_FLOW_STATUS } from 'graphql/mutations/update-flow-status';
 import { UPDATE_FLOW } from 'graphql/mutations/update-flow';
@@ -111,18 +112,23 @@ export default function EditorLayout() {
         </Box>
 
         <Box pr={1}>
-          <Button
-            variant="contained"
-            size="small"
-            onClick={() => onFlowStatusUpdate(flow ? !flow.active : false)}
-            data-test={
-              flow?.active ? 'unpublish-flow-button' : 'publish-flow-button'
-            }
-          >
-            {flow?.active
-              ? formatMessage('flowEditor.unpublish')
-              : formatMessage('flowEditor.publish')}
-          </Button>
+          <Can I="publish" a="Flow" passThrough>
+            {(allowed) => (
+              <Button
+                disabled={!allowed || !flow}
+                variant="contained"
+                size="small"
+                onClick={() => onFlowStatusUpdate(!flow.active)}
+                data-test={
+                  flow?.active ? 'unpublish-flow-button' : 'publish-flow-button'
+                }
+              >
+                {flow?.active
+                  ? formatMessage('flowEditor.unpublish')
+                  : formatMessage('flowEditor.publish')}
+              </Button>
+            )}
+          </Can>
         </Box>
       </TopBar>