Support React 18 about react-spinners

[adrianmanduc]

Any update on this ?

[RubenVerster]

Use this, please
npm install --save react-spinners --legacy-peer-deps

You might have to run this as well
npm i react-dom --legacy-peer-deps

And just to be safe, run
npm i

Do you guys like not google or check your consoles or something...

[davidhu2000]

v0.12.0 is released with react 18 support

[nikeee]

Yes, that might work. However, as seen in your instructions, --legacy-peer-deps uses a legacy mode, which should be avoided.
Fixing this upstream would be appropriate.

There is a PR for upgrading the dependency: #464

[davidhu2000]

try 0.12.0-alpha.3 and see if the issue is resolved

[davidhu2000]

oof, this might be more compcliated than expected given enzyme doesn't have an adapter for react 18

[RubenVerster]

Yes, that might work. However, as seen in your instructions, --legacy-peer-deps uses a legacy mode, which should be avoided. Fixing this upstream would be appropriate.

There is a PR for upgrading the dependency: #464

I know man... but this is the current fix for the issue right now
Once the PR has been merged in, they will come and tell us that the issue has an official fix :P

[aperalta1431]

I appreciate the above solution but it should be avoided. Besides the above solution involves making changes to the pipeline to support such legacy mode if we still want to proceed with it which is not suitable in an enterprise setting. Nonetheless, the library is a time saver and I will be waiting for an update on this.

[Ali-Aref]

any updates ?