fix: ensure all fields are available in final completion event

FrigadeHQ · Aug 23, 2024 · b6438e4 · b6438e4
1 parent 0f5c086
commit b6438e4
Show file tree

Hide file tree

Showing 3 changed files with 15 additions and 2 deletions.
diff --git a/.changeset/bright-mice-change.md b/.changeset/bright-mice-change.md
@@ -0,0 +1,6 @@
+---
+"@frigade/js": patch
+"@frigade/react": patch
+---
+
+Adds the entire form context to flow completion event
diff --git a/packages/js-api/src/core/flow.ts b/packages/js-api/src/core/flow.ts
@@ -265,7 +265,7 @@ export class Flow extends Fetchable {
           thisStep.id
         )
         if (isLastStep) {
-          await this.sendFlowStateToAPI(COMPLETED_FLOW)
+          await this.sendFlowStateToAPI(COMPLETED_FLOW, properties)
         }
       }
 
@@ -487,6 +487,13 @@ export class Flow extends Fetchable {
       .indexOf(currentStep)
   }
 
+  /**
+   * Get the progress of the flow as a number between 0 and 1. Useful when rendering a progress bar.
+   */
+  public getProgress(): number {
+    return (this.getNumberOfCompletedSteps() || 0) / (this.getNumberOfAvailableSteps() || 1)
+  }
+
   /**
    * @ignore
    */

diff --git a/packages/react/src/components/Form/FormStep.tsx b/packages/react/src/components/Form/FormStep.tsx
@@ -108,7 +108,7 @@ export function FormStep({
   const disabled = step.$state.blocked || !formContext.formState.isValid ? true : false
 
   useEffect(() => {
-    formContext.reset()
+    formContext.clearErrors()
   }, [step])
 
   return (