-
Notifications
You must be signed in to change notification settings - Fork 23
React 19 upgrade #5552
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 19 upgrade #5552
Conversation
🦋 Changeset detectedLatest commit: 19acc98 The changes in this PR will be included in the next version bump. This PR includes changesets to release 4 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
A bunch of tests are failing. I reckon it might be because of mismatch react version from possibly storybook? I also noticed react and react dom as dependency in the root package.json - are they needed there? |
7dd6d47
to
7a6805f
Compare
a31c6ed
to
5e409ed
Compare
@@ -43,6 +43,7 @@ $cell-border-radius: 3px; | |||
justify-content: space-between; | |||
width: 100%; | |||
color: $color-purple-800; | |||
top: -0.25rem; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes look good but given how much the deps update for react day picker impact the behavior of FilterDateRangePicker and DateRangePicker, can we update or add another changeset with a patch to call out this update out specifically. It'll help if we ever need to refer to the changelog cc: @ryanseddon
|
* chore: Ignore peerDep warnings for older packages Target specific packages so as not to ignore valid warnings we can ignore * chore: add react popper * chore: add popper core * chore: add changeset and order packages
* fix: remove root-level React dependencies * Add react as a peerDep and don't rely on root * Add a devDep too to match other packages
* feat(components): upgrade react-day-picker from 8.10.1 to 9.6.7 Update react-day-picker to version 9.6.7 and update all dependent components to work with the new API. Changes include: - Refactor component props to use the updated type definitions - Update component rendering to match the new API requirements - Modify CSS modules to accommodate new class naming conventions - Adjust navigation components to use the new Chevron component API - Add new styles for proper range highlighting and hidden elements * update test to target button correctly * fix linting problems * Failing tests * fix(Calendar): replace ClassNames type casting with explicit PropsBase typing * Fix classnames * Fix css changes and first child not working in v9 dom
* fix: use ts-patch before building components * chore: changeset
* fix: remove root-level React dependencies * Add react as a peerDep and don't rely on root * Add a devDep too to match other packages
* fix: use ts-patch before building components * chore: changeset
* fix: remove root-level React dependencies * Add react as a peerDep and don't rely on root * Add a devDep too to match other packages
* fix: use ts-patch before building components * chore: changeset
* fix: use ts-patch before building components * chore: changeset
- Add top spacing (-0.25rem) to navigation elements in all Calendar variants - Fix styling for today's date and disabled days - Update test selectors to properly target day buttons within grid cells
00d0ef4
to
19acc98
Compare
Important: Request PR reviews on Slack
Please reach out to the design system team on Slack in
#prod_design_system
for PR reviews. GitHub notifications (e.g. from tagging a person) are not actively monitored.Why
To prepare for our Nextjs 15 and React 19 upgrade, I created this PR to test it out. My findings has been documented here
What