update flow config upon onClose or onSubmit

Author: m0nggh

package-lock.json

@@ -11,6 +11,7 @@ type Params = {
     id: string
     notificationFrequency: IFlowErrorConfig['notificationFrequency']
     hasLoadedOnce: IFlowDemoConfig['hasLoadedOnce']
+    showSurvey: boolean
   }
 }
 
@@ -38,11 +39,15 @@ const updateFlowConfig = async (
   }
 
   // TODO (mal): remove demo config
-  if (params.input.hasLoadedOnce !== undefined) {
-    newConfig.demoConfig = {
-      ...newConfig.demoConfig, // If ever undefined (should never be), it gets set to an empty object first
-      hasLoadedOnce: params.input.hasLoadedOnce,
-    }
+  // if (params.input.hasLoadedOnce !== undefined) {
+  //   newConfig.demoConfig = {
+  //     ...newConfig.demoConfig, // If ever undefined (should never be), it gets set to an empty object first
+  //     hasLoadedOnce: params.input.hasLoadedOnce,
+  //   }
+  // }
+
+  if (params.input.showSurvey !== undefined) {
+    newConfig.showSurvey = params.input.showSurvey
   }
 
   return await flow.$query().patchAndFetch({

packages/backend/src/graphql/mutations/update-flow-config.ts

@@ -43,7 +43,7 @@ type Query {
   getTableConnections(tableIds: [String!]!): JSONObject
   getTables(limit: Int!, offset: Int!, name: String): PaginatedTables!
   # Tiles rows
- getAllRows(tableId: String!, stringifiedCursor: String): GetTableRowsResult!
+  getAllRows(tableId: String!, stringifiedCursor: String): GetTableRowsResult!
   getCurrentUser: User
   healthcheck: AppHealth
   getPlumberStats: Stats
@@ -450,6 +450,7 @@ input UpdateFlowConfigInput {
   id: String!
   notificationFrequency: NotificationFrequency
   hasLoadedOnce: Boolean
+  showSurvey: Boolean
 }
 
 input ExecuteFlowInput {

packages/backend/src/graphql/schema.graphql

@@ -41,7 +41,7 @@
     "dedent": "1.5.1",
     "escape-html": "1.0.3",
     "file-saver": "2.0.5",
-    "lens-widget": "1.35.0",
+    "lens-widget": "1.36.0",
     "lodash": "^4.17.21",
     "lodash.get": "4.4.2",
     "lottie-web": "5.12.2",

packages/frontend/package.json

@@ -23,6 +23,7 @@ import {
   StepExecutionsToIncludeProvider,
 } from '@/contexts/StepExecutionsToInclude'
 import { CREATE_STEP } from '@/graphql/mutations/create-step'
+import { UPDATE_FLOW_CONFIG } from '@/graphql/mutations/update-flow-config'
 import { UPDATE_STEP } from '@/graphql/mutations/update-step'
 import { GET_APPS } from '@/graphql/queries/get-apps'
 import { GET_FLOW } from '@/graphql/queries/get-flow'
@@ -103,6 +104,13 @@ export default function Editor(props: EditorProps): React.ReactElement {
   const showSurvey = flow.active && flow.config?.showSurvey
   const { currentUser } = useAuthentication()
 
+  const [updateFlowConfig] = useMutation(UPDATE_FLOW_CONFIG)
+  const onFlowConfigUpdate = useCallback(async () => {
+    await updateFlowConfig({
+      variables: { input: { id: flow.id, showSurvey: false } },
+    })
+  }, [updateFlowConfig, flow.id])
+
   const steps = useMemo(
     // Populate each step's flowId so that IStep isn't LYING about flowId being
     // non-undefined. We do it here instead of fetching in GraphQL since all
@@ -310,6 +318,8 @@ export default function Editor(props: EditorProps): React.ReactElement {
             `UserEmail: ${currentUser?.email}`,
             ...(appConfig.env !== 'prod' ? [`Env: ${appConfig.env}`] : []),
           ]}
+          onSubmit={onFlowConfigUpdate}
+          onClose={onFlowConfigUpdate}
         />
       )}
     </Flex>