-
Notifications
You must be signed in to change notification settings - Fork 98
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
(fix) Peer dependencies issue related to React v18 #191
base: master
Are you sure you want to change the base?
Conversation
Any chance this could be merged? |
@schiehll we're held up on this to upgrade our app, can we merge? 🙏 |
@DavidChouinard until this PR is merged you can try out it's fork with the React 18 support - https://www.npmjs.com/package/@blaumaus/react-alert |
Fantastic thank you. @schiehll would still be great to have this merged to have an official supported distribution |
@schiehll merge it pls |
@schiehll when is the merge? |
https://www.npmjs.com/package/@blaumaus/react-alert - this is not working and has the same issue as the original Node Package. |
poke |
Any reason this hasn't been merged yet? |
maybe @schiehll is no longer with us |
This PR fixed the issue occurring in some environments while installing this package.