Compatibility with StrictMode

[piotrpospiech]

This fixes #1194.

Overriding setState caused this issue. I removed it and added checking if the component is mounted before calling setState.

Tested with Vite, React v18.2.0 and Strict Mode.

[codecov]

Codecov Report

Attention: Patch coverage is 58.33333% with 5 lines in your changes are missing coverage. Please review.

Project coverage is 94.43%. Comparing base (4b3f109) to head (e76fe3d).

Files	Patch %	Lines
packages/uniforms/src/BaseForm.tsx	58.33%	0 Missing and 5 partials ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master    #1324      +/-   ##
==========================================
- Coverage   94.55%   94.43%   -0.13%     
==========================================
  Files         231      231              
  Lines        3823     3827       +4     
  Branches     1030     1035       +5     
==========================================
- Hits         3615     3614       -1     
  Misses         82       82              
- Partials      126      131       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[mdesousa]

hi @piotrpospiech @zendranm thanks for this fix! wondering if there are plans to patch it in version 3?

[piotrpospiech]

Hi @mdesousa, we didn't consider it. Our focus is on version 4, which is nearly finished (v4.0 Milestone) and mainly needs documentation updates. We don't plan more changes to v3 packages. If this fix is important for you, I'd recommend trying the v4 beta or waiting for the full v4 release coming soon.

[mdesousa]

@piotrpospiech unfortunately our app is in production and can't upgrade to a beta library yet. We'd also expect that there will be breaking changes and more testing necessary to upgrade to 4.0... which we'll happily do when it's ready. But in the meantime getting this small patch for version 3 would help us with this issue that is impacting our production app. Thanks!