Repository navigation
Replies: 2 comments 7 replies
|
Hm, it sounds like a bug. Would anyone like to look into it? (Yeah, |
|
This still reproduces on current main (
The persist wrapper forwards only the first two arguments to the saved setter, dropping the action argument. I noticed kelvin-du's earlier offer. I have a minimal argument-forwarding fix and regression prepared privately: all 225 tests, formatting, type checks, lint and build pass on macOS. The build emits a TS5096 configuration warning. Codex prepared the change and an additional independent Codex review found no blocker; no human review is claimed. Is the earlier work still in progress, or would maintainers like the focused branch shared here under the current contribution workflow? |
Uh oh!
There was an error while loading. Please reload this page.
Hi, I tried to use "no actions store" approach to implement my app logic. Everything worked correctly, but it didn't work with redux-devtool. The actions in devtool showed "anonymous" instead of the actual action name . Then I tried to put the actions back in the store state, it worked again.
Here is the simplified version:
Then
updateUserwas called in a component.Also, can I put
falseinto the second param of thesetandsetStateinstead ofundefined(I don't want to override the default merging behavior, just don't likeundefined)?All reactions