[BUG] Make useForm success redirect optional

[pcfreak30]

Describe the bug

refine/packages/core/src/hooks/form/index.ts

Line 241 in 4d3b671

if (!isPessimistic && !isAutosave) {

refine/packages/core/src/hooks/form/index.ts

Line 291 in 4d3b671

if (isPessimistic && !isAutosave) {

It is assumed that on success you want to always redirect. This should be made an opt-out. I use refine as a data abstraction layer and this is forcing a redirect after submitting a dialog form, away from the current page, which is not wanted.

Steps To Reproduce

n/a

Expected behavior

Don't redirect on success.

Packages

• @refinecore/dev

Additional Context

No response

[DexAsHisH]

Hey, I would love to work on this bug issue. Could you assign this to me?

[BatuhanW]

Hey @pcfreak30 thanks for the issue.

Changing the default behaviour would be a breaking change and require a discussion first. However, maybe we can allow it to be disabled globally.

[pcfreak30]

Hey @pcfreak30 thanks for the issue.

Changing the default behaviour would be a breaking change and require a discussion first. However, maybe we can allow it to be disabled globally.

please re-read. I didn't say change global behavior, I asked for an opt-out option. Ex.

if (!isPessimistic && !isAutosave) {
        // If the mutation mode is not pessimistic and redirect is not disabled,
        // handle the redirect immediately in an async manner
        if (props.redirectOnSuccess !== false) {
          // `setWarnWhen` blocks the redirects until set to `false`
          // If redirect is done before the value is properly set, it will be blocked.
          // We're deferring the execution of the redirect to ensure that the value is set properly.
          deferExecution(() => onSuccessRedirect());
        }
        // Resolve the promise immediately
        resolve();
      }

[sidshas03]

Hi, I’d like to work on this issue.
I’ll add a new optional prop redirectOnSuccess?: boolean to useForm. By default it will stay true (so no breaking change), but when set to false it will skip the automatic redirect in both pessimistic and non-pessimistic flows.
Please assign this to me.

[NithinSajjanam]

feat(useForm): add opt-out redirectOnSuccess prop to prevent automatic redirect

Previously, useForm always redirected after a successful form submission.
This change introduces an explicit redirectOnSuccess prop.

If redirectOnSuccess is set to false, automatic redirection after form submission will be skipped.

Example usage:
useForm({ redirectOnSuccess: false });

This is useful for cases like dialog forms where staying on the same page is desired.