Skip to content

Allow React 19 in peer deps#118

Merged
david-crespo merged 5 commits into
masterfrom
react-19
Feb 18, 2025
Merged

Allow React 19 in peer deps#118
david-crespo merged 5 commits into
masterfrom
react-19

Conversation

@david-crespo

Copy link
Copy Markdown
Collaborator

Just changing the peer deps might be insufficient. We might need to bump stuff like @headlessui/react.

Comment thread package.json Outdated
@david-crespo david-crespo mentioned this pull request Feb 17, 2025
8 tasks
@david-crespo

Copy link
Copy Markdown
Collaborator Author

I tested the canary on the marketing site, which is on React 18, to make sure it doesn't only work with React 19. It all seemed to work fine.

Comment thread package.json
"@floating-ui/react": "^0.25.1",
"@headlessui/react": "^1.7.17",
"@floating-ui/react": "^0.27.4",
"@headlessui/react": "^2.2.0",

@david-crespo david-crespo Feb 17, 2025

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the only major bump, so it's worth testing specifically. It's used in the Listbox, which is only used in one spot: the marketing site events page. It works fine.

https://github.com/search?q=org%3Aoxidecomputer+design-system+listbox+NOT+repo%3Aoxidecomputer%2Fconsole&type=code

https://github.com/oxidecomputer/oxide-computer/blob/f83e368dc899e9cca6bb1cf168c72befc153dbce/app/routes/events.tsx#L64-L70

image

@david-crespo david-crespo merged commit 5e70f6e into master Feb 18, 2025
@david-crespo david-crespo deleted the react-19 branch February 18, 2025 18:31
@benjaminleonard

Copy link
Copy Markdown
Collaborator

🚀 PR was released in v2.2.2 🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants