react-final-form-listeners Bump peerDependency react to include 18.0.0

Bump peerDependency react to include 18.0.0

Open simian-loco opened this issue 2 years ago • 8 comments

Resolves issue #41

Feb 08 '23 17:02 simian-loco

@erikras - Could you review and merge this? Would appreciate it very much :)

Mar 14 '23 16:03 mikaelengstrom

@mikaelengstrom @erikras -- This change eventually broke for me. It fixed my build and I only had a couple of implementations so I opened this merge up; but I did find that it eventually broke. I can't exactly remember what it was that broke; but I think it was a react API call somewhere which was incompatible with React 18. I ended up just refactoring out the two listener implementations I was using and removed this package from my dependencies.

I would close this merge as breaking; but perhaps this might be a good place for related discussion? @mikaelengstrom could you bump locally in your node_modules/react-final-form-listeners/package.json and see which react API call this breaks on? I think I was using onChange.

Edit: added some details

05/05/23 Update - I'll defer to others here regarding everything working as expected as I cannot reproduce the issue I mentioned above at this time and it was a pretty convoluted abstraction in the first place where the error was occurring.

Mar 14 '23 17:03 simian-loco

@simian-loco seems to work just fine in our app

Mar 20 '23 15:03 mikaelengstrom

@erikras may you please consider merging this? We're maintaining pretty large project on React@18 with 50+ usages of library in huge forms. And still everything works as expected.