issue: Fields dirty state is not updated when passing values to useForm since 7.51.0

[djhi]

Version Number

7.51.0

Codesandbox/Expo snack

https://codesandbox.io/p/sandbox/control-forked-stnhsf

Steps to reproduce

To reproduce, change the value of the only input and observe the isDirty value displayed by the IsolateReRender component. It stays false

Expected behaviour

It should be set to true.

I actually set up two sandboxes:

• with v7.50.1: https://codesandbox.io/p/sandbox/control-forked-sjggjc
• with v7.51.0: https://codesandbox.io/p/sandbox/control-forked-stnhsf

The issue is still present in v7.51.3.

This breaks all react-admin applications.

What browsers are you seeing the problem on?

No response

Relevant log output

No response

Code of Conduct

• I agree to follow this project's Code of Conduct

[bluebill1049]

thanks, for the report, looks like the issue is already fixed: https://codesandbox.io/p/sandbox/control-forked-5fys45?file=%2Fsrc%2FApp.tsx%3A9%2C20 related issue: #11778

I will release a patch for this soon.

[djhi]

There's something I don't understand though. My second sandbox (https://codesandbox.io/p/sandbox/control-forked-stnhsf) uses 7.51.3 just like yours, yet it has the same issue but yours don't. Besides, I don't see any differences between the two.

[slax57]

There's something I don't understand though. My second sandbox (https://codesandbox.io/p/sandbox/control-forked-stnhsf) uses 7.51.3 just like yours, yet it has the same issue but yours don't. Besides, I don't see any differences between the two.

That's because the fixed sandbox imports from ./src and not from react-hook-form

[djhi]

There's something I don't understand though. My second sandbox (https://codesandbox.io/p/sandbox/control-forked-stnhsf) uses 7.51.3 just like yours, yet it has the same issue but yours don't. Besides, I don't see any differences between the two.

That's because the fixed sandbox imports from ./src and not from react-hook-form

Good catch, thanks :)

[djhi]

@bluebill1049 Any chance to have a release soon? 🙏

[bluebill1049]

@bluebill1049 Any chance to have a release soon? 🙏

yes, sorry I am on a break.

[djhi]

Hi @bluebill1049! Thank you for the release. It does not seem to be fixed though 😭
https://codesandbox.io/p/sandbox/control-forked-stnhsf

[TheWirv]

I have breaking unit tests because a CSS class is applied based on the isDirty state, and I check in the tests, whether the class is applied or not.

I have logged some of the data while rendering the component, and this is the final state, with version 7.51.4:

I have executed the same test on an older version (I am trying to upgrade package versions for our project right now) and there it works fine, with version 7.15.4:

Both use the same userEvent.type API from the user event testing library.