Better fix for workflow action reactivity (baserow#4242)

Author: jrmi

web-frontend/modules/builder/components/event/WorkflowAction.vue

@@ -105,7 +105,7 @@ export default {
         await this.actionUpdateWorkflowAction({
           page: this.elementPage,
           workflowAction: this.workflowAction,
-          values,
+          values: differences,
         })
       } catch (error) {
         this.$refs.actionForm?.reset()

web-frontend/modules/builder/store/builderWorkflowAction.js

@@ -60,21 +60,19 @@ const mutations = {
       order,
     } = page.workflowActions[index]
 
-    const newValue = overwrite
-      ? populateWorkflowAction({
-          id,
-          page_id: pageId,
-          element_id: elementId,
-          event,
-          order,
-          ...values,
-        })
-      : {
-          ...page.workflowActions[index],
-          ...values,
-        }
-
-    page.workflowActions.splice(index, 1, newValue)
+    if (overwrite) {
+      const newValue = populateWorkflowAction({
+        id,
+        page_id: pageId,
+        element_id: elementId,
+        event,
+        order,
+        ...values,
+      })
+      page.workflowActions.splice(index, 1, newValue)
+    } else {
+      Object.assign(page.workflowActions[index], values)
+    }
   },
   SET_ITEM(state, { page, workflowAction: workflowActionToSet, values }) {
     page.workflowActions = page.workflowActions.map((workflowAction) =>

web-frontend/modules/core/components/formula/FormulaInputField.vue

@@ -315,9 +315,7 @@ export default {
     nodesHierarchy(newValue, oldValue) {
       // fixes reactivity issue with components in tiptap by forcing the input to
       // render.
-      if (!_.isEqual(newValue, oldValue)) {
-        this.key += 1
-      }
+      this.key += 1
     },
     disabled(newValue) {
       this.editor.setOptions({ editable: !newValue && !this.readOnly })