-
Notifications
You must be signed in to change notification settings - Fork 32
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
React 17 update? #37
Comments
Or even react 18 at this point |
I think it works with any react version, is npm refusing to install because of the peer dependency? |
I've created a PR to update for React 18: #39. Didn't support 17 tho, but feel free to comment on it. @benwiley4000 The v0.4.2 built files did work with React 18, but the whole build/dev pipeline didn't, which makes it difficult to install in another project, so I made quite a few upgrades to the dependencies too. |
Thanks, I responded in the PR.
…On Wed, Jan 24, 2024 at 6:52 AM Tak Tran ***@***.***> wrote:
I've created a PR to update for React 18: #39
<#39>. Didn't
support 17 tho, but feel free to comment on it.
@benwiley4000 <https://github.com/benwiley4000> The v0.4.2 built files
did work with React 18, but the whole build/dev pipeline didn't, which
makes it difficult to install in another project, so I made quite a few
upgrades to the dependencies too.
—
Reply to this email directly, view it on GitHub
<#37 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ADHOD3M4JMDQJG7NZZ2JEP3YQDYXFAVCNFSM5MFHMEXKU5DIOJSWCZC7NNSXTN2JONZXKZKDN5WW2ZLOOQ5TCOJQG44TONRVHAYQ>
.
You are receiving this because you were mentioned.Message ID:
***@***.***>
|
Hey there!
Are you going to update this package to work with react 17?
Thanks in advance!
The text was updated successfully, but these errors were encountered: