Using container on update hook causes React 16 errors to be printed

[yamadapc]

Using container on update hook causes React 16 errors to be printed

[albertogasparin]

Marked as duplicated because it has the same root cause: the conversion to functional component. By going back to class and using getDerivedStateFromProps to trigger those state changes, the warning and the deoptimisation go away (could not reproduce anymore).
I assume because that static method is called before render phase and so mutating state is still a valid operation

[albertogasparin]

Duplicated #219
We have a PR up #221 that however needs validation

[yamadapc]

Is it duplicated?

[yamadapc]

This isn't really a "change in behaviour caused by React 17 to 18", but it is a bug in React 16 where using the setState API in onUpdate will always trigger an error to be logged.

This is a problem because developers will be confused by the warnings being logged and not know that they are coming from React Sweet State.

This is still an issue with certain options after #221.

I suggest the setState parameter be removed from the update hook API on the cases it will break / print a warning.

[albertogasparin]

While we work on #221 , a workaround is to set

defaults.batchUpdates = true

which is what we use across some products in production