feat: pass the changed attribute though the onChange form callback

Ref: #143
AdeAttwood · Mar 25, 2023 · e751896 · e751896 · vercel · Mar 25, 2023
1 parent 7682422
commit e751896
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 2 deletions.
diff --git a/src/form.tsx b/src/form.tsx
@@ -61,7 +61,7 @@ export interface FormProps<T extends {}> {
   /**
    * Callback that is called whenever an attribute is changed
    */
-  onChange?: (context: ReturnType<Form<T>["getContextValue"]>) => void;
+  onChange?: (context: ReturnType<Form<T>["getContextValue"]>, attribute: string) => void;
   /**
    *
    * TODO(ade): sort out prop with children
@@ -193,7 +193,7 @@ export class Form<T extends Record<string, any>> extends React.Component<FormPro
     }
 
     this.setState({ formState, errors, status });
-    this.props.onChange?.(this.getContextValue());
+    this.props.onChange?.(this.getContextValue(), attribute);
   };
 
   /**

diff --git a/tests/on-change.spec.tsx b/tests/on-change.spec.tsx
@@ -22,4 +22,5 @@ it("will call onChange when a attribute is changed", async () => {
   });
 
   expect(onChange).toHaveBeenCalled();
+  expect(onChange).toBeCalledWith(expect.objectContaining({ formState: { "test-input": "A" } }), "test-input");
 });