React Admin: onSuccess does not work properly on Component

Question

onSuccess function does not work properly on react-admin,

my code:

const onSuccess = () => {
   redirect('list', props.basePath);
};


<Edit 
  onFailure={onFailure}  
  onSuccess={onSuccess}  
  title="Ediar Usuário" 
  {...props} 
>
  <SimpleForm 
     variant="standard" 
     toolbar={<CustomToolbar />} 
  >
</Edit>

On the first time, it works perfectly but at second time, nothing happens.

Do not even trigger the save event

Hi, would you kindly explain what you are trying to achieve? By default, once edit is finished the form redirects to the list view. — MwamiTovi, Jul 17 '20 at 14:42

score 3 · Answer 1 · answered Jul 30 '20 at 08:01

3

I don't know if this is applicable to your use case but setting undoable to false on Edit component works

answered Jul 30 '20 at 08:01

Jasper Bernales

1,601
1
11
16

Hyungju Moon · Answer 2 · 2020-08-10T05:57:54.943

I am suffering from the same problem.

Jasper Bernales's tip was effective.

I've changed my code from :

<Edit
      onSuccess={onSuccess}
      {...props}
    >

to :

<Edit
      onSuccess={onSuccess}
      undoable={false}
      {...props}
    >

then ...it works!

It seems like a problem caused by forcing "useRedirect" or "useRefresh" to interfere with the delay scheduled by "undoable".The document seems to need an update to this part. just check this out from React-Admin Docs.:

You can disable this behavior by setting undoable={false}. With that setting, clicking on the Delete button displays a confirmation dialog. Both the Save and the Delete actions become blocking and delay the refresh of the screen until the data provider responds.

score 0 · Answer 3 · answered May 18 '21 at 14:07

Having the same issue I came across this issue on the react-admin github page. So basically it is not a bug but ... kinda feature the way the Edit component works. In short:

const onSuccess = () => {
     ...
     notify('success', 'info', null, true);
     ...
}

This is the way you should call notify inside your custom onSuccess function to trigger the real update on the dataProvider.

React Admin: onSuccess does not work properly on Component

3 Answers3