fix: Ensure flowId is populated on settings routes

theopensystemslab · Sep 23, 2024 · e9c70e3 · e9c70e3
1 parent b6c2b72
commit e9c70e3
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 1 deletion.
diff --git a/editor.planx.uk/src/pages/FlowEditor/lib/store/shared.ts b/editor.planx.uk/src/pages/FlowEditor/lib/store/shared.ts
@@ -17,6 +17,7 @@ export interface SharedStore extends Store.Store {
   flowName: string;
   flowAnalyticsLink: string | null;
   id: string;
+  setFlowId: (id: string) => void;
   getNode: (id: NodeId) => Store.Node | undefined;
   resetPreview: () => void;
   setFlow: ({
@@ -63,6 +64,9 @@ export const sharedStore: StateCreator<
   flowAnalyticsLink: null,
 
   id: "",
+
+  setFlowId: (id) => set({ id }),
+
   previewEnvironment: "standalone",
 
   setPreviewEnvironment(previewEnvironment: PreviewEnvironment) {

diff --git a/editor.planx.uk/src/routes/serviceSettings.tsx b/editor.planx.uk/src/routes/serviceSettings.tsx
@@ -19,7 +19,7 @@ interface GetFlowSettings {
 export const getFlowSettings = async (req: NaviRequest) => {
   const {
     data: {
-      flows: [{ settings, status }],
+      flows: [{ id, settings, status }],
     },
   } = await client.query<GetFlowSettings>({
     query: gql`
@@ -40,6 +40,7 @@ export const getFlowSettings = async (req: NaviRequest) => {
     },
   });
 
+  useStore.getState().setFlowId(id);
   useStore.getState().setFlowSettings(settings);
   useStore.getState().setFlowStatus(status);
 };