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
[Bug]: Error when using with Immer 10 #312
Comments
Would it make sense to update the version of |
Could you try it with version |
👍 Will do and then let you know |
It works great on the codesandbox, will try on our app locally then edit this comment: https://codesandbox.io/p/sandbox/heuristic-firefly-4y96td EDIT: On our local application, I'm seeing this error in the page where the form is used. I'll try to dig into it and provide a reproduction:
|
Hi! Sorry for the slow turnaround. Could you post a reproduction of the new error so I can debug it? |
The initial fix is now released in 5.1.2. |
Nevermind, I've managed to reproduce the issue. I've rolled back the initial fix in 5.1.3 because of the dynamic require issue. |
I wasn't able to reproduce the issue in a codesandbox and could not figure out why it was failing in our local app, so we've moved forward with locking our own version of immer for now. Do you have a sandbox where you managed to reproduce the issue? I'd be curious as to what caused it. |
Which packages are impacted?
remix-validated-form
@remix-validated-form/with-zod
@remix-validated-form/with-yup
zod-form-data
What version of these packages are you using?
remix-validated-form
:^5.0.1
@remix-validated-form/with-zod
:^2.0.6
immer
:^10.0.2
Please provide a link to a minimal reproduction of the issue.
https://codesandbox.io/p/sandbox/amazing-frost-2g4gy4
Steps to Reproduce the Bug or Issue
Expected behavior
I would expect the library to work with any version of immer installed locally, since the library should export its own version of immer? Seems like NPM is picking up the locally installed version instead, and the library does not support v10.
Screenshots or Videos
No response
Platform
Additional context
No response
The text was updated successfully, but these errors were encountered: