Skip to content
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: lock jotai version to 1.4.0 #555

Conversation

marcinciarka
Copy link

The current "jotai": "^1.3.7" works well when testing with the react-cosmos stack cause it's using the locked version from the project yarn.lock file (which is 1.4.0). When using the widget somewhere else (eg. anywhere tbh) it's using the newer version (1.13.1) which is not compatible.

This PR makes sure that the proper version (exact same as when running the project from this repo) is used and will resolve this issue: #374

@vercel
Copy link

vercel bot commented Mar 13, 2023

@marcinciarka is attempting to deploy a commit to the Uniswap Team on Vercel.

A member of the Team first needs to authorize it.

@marcinciarka marcinciarka changed the title Lock jotai version to 1.4.0 fix: lock jotai version to 1.4.0 Mar 13, 2023
@vercel
Copy link

vercel bot commented Mar 13, 2023

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated
widgets ✅ Ready (Inspect) Visit Preview 💬 Add your feedback Mar 13, 2023 at 5:11PM (UTC)

@vm
Copy link
Contributor

vm commented Mar 13, 2023

we'll re-run the tests and then merge this 👍

@vercel vercel bot temporarily deployed to Preview March 13, 2023 17:11 Inactive
@vm vm mentioned this pull request Mar 13, 2023
@vm
Copy link
Contributor

vm commented Mar 13, 2023

https://github.com/Uniswap/widgets/releases/tag/v2.47.10 should solve your issue. unfortunately, this branch's tests keep failing for an unknown reason, so i merged the same diff in #556. we'll figure that out separately. thanks for the effort here!

@vm vm closed this Mar 13, 2023
@marcinciarka marcinciarka deleted the marcinciarka/lock-jotai-version branch March 13, 2023 23:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants