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: react 19 compatibility #63

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

fix: react 19 compatibility #63

wants to merge 2 commits into from

Conversation

rexxars
Copy link
Member

@rexxars rexxars commented Dec 14, 2024

This PR:

  • Drops @sanity/icons from dependencies, since it causes multiple versions of @sanity/icons to be installed, and the 1.x range is not compatible with React 19, and we were only using two icons from it (they are now inlined)
  • Drops react-copy-to-clipboard in favor of just calling navigator.clipboard.writeText(), which removes another React 19 incompatibility
  • Drops unused dev dependencies
  • Bumps the peer dependency range to include React 19

@rexxars rexxars requested a review from a team as a code owner December 14, 2024 01:56
Copy link

vercel bot commented Dec 14, 2024

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

Name Status Preview Comments Updated (UTC)
incompatible-plugin ✅ Ready (Inspect) Visit Preview 💬 Add feedback Dec 14, 2024 1:56am

Copy link

New and removed dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher

🚮 Removed packages: npm/@types/react-copy-to-clipboard@5.0.4, npm/@types/styled-components@5.1.26

View full report↗︎

@rexxars rexxars requested review from snorrees and stipsan December 14, 2024 06: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.

1 participant