chore: onMetaChange should trigger on unmount

Author: zombieJ

src/Field.tsx

@@ -76,7 +76,7 @@ export interface InternalFieldProps<Values = any> {
   messageVariables?: Record<string, string>;
   initialValue?: any;
   onReset?: () => void;
-  onMetaChange?: (meta: Meta) => void;
+  onMetaChange?: (meta: Meta & { destroy?: boolean }) => void;
   preserve?: boolean;
 
   /** @private Passed by Form.List props. Do not use since it will break by path check. */
@@ -168,6 +168,7 @@ class Field extends React.Component<InternalFieldProps, FieldState> implements F
 
   public componentWillUnmount() {
     this.cancelRegister();
+    this.triggerMetaEvent(true);
     this.mounted = false;
   }
 
@@ -215,10 +216,10 @@ class Field extends React.Component<InternalFieldProps, FieldState> implements F
     }));
   };
 
-  public triggerMetaEvent = () => {
+  public triggerMetaEvent = (destroy?: boolean) => {
     const { onMetaChange } = this.props;
 
-    onMetaChange?.(this.getMeta());
+    onMetaChange?.({ ...this.getMeta(), destroy });
   };
 
   // ========================= Field Entity Interfaces =========================

tests/index.test.js

@@ -217,6 +217,20 @@ describe('Form.Basic', () => {
       expect(form.getFieldError('password')).toEqual(["'password' is required"]);
       expect(form.isFieldTouched('password')).toBeTruthy();
     });
+
+    it('remove Field should trigger onMetaChange', () => {
+      const onMetaChange = jest.fn();
+      const wrapper = mount(
+        <Form>
+          <Field name="username" onMetaChange={onMetaChange}>
+            <Input />
+          </Field>
+        </Form>,
+      );
+
+      wrapper.unmount();
+      expect(onMetaChange).toHaveBeenCalledWith(expect.objectContaining({ destroy: true }));
+    });
   });
 
   it('should throw if no Form in use', () => {