Skip to content

feat: Update react virtual to tanstack react virtual #3312

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

Merged

Conversation

mannycarrera4
Copy link
Contributor

@mannycarrera4 mannycarrera4 commented May 27, 2025

Summary

Fixes: #3311
Replaces react-virtual v2 with @tanstack/react-virtual v3

Release Category

Components

Release Note

UNSTABLE_virtual is now a reference to @tanstack/react-virtual. Visit Tanstack docs for a full list of properties.


Copy link

cypress bot commented May 27, 2025

Workday/canvas-kit    Run #8889

Run Properties:  status check passed Passed #8889  •  git commit 063469b5f1 ℹ️: Merge 198e774ac6b8e29c821eeb1b841453adc1b610c5 into 3aa2cc1a8933ef332196a2ed0202...
Project Workday/canvas-kit
Branch Review mc-update-react-virtual
Run status status check passed Passed #8889
Run duration 02m 52s
Commit git commit 063469b5f1 ℹ️: Merge 198e774ac6b8e29c821eeb1b841453adc1b610c5 into 3aa2cc1a8933ef332196a2ed0202...
Committer Manuel Carrera
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 21
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 936
View all changes introduced in this branch ↗︎
UI Coverage  21.35%
  Untested elements 1523  
  Tested elements 411  
Accessibility  99.28%
  Failed rules  6 critical   5 serious   0 moderate   2 minor
  Failed elements 98  

@mannycarrera4 mannycarrera4 added the ready for review Code is ready for review label May 30, 2025
@mannycarrera4 mannycarrera4 moved this to 👀 In review in Canvas Kit May 30, 2025
@mannycarrera4 mannycarrera4 added this to the 13.1 milestone May 30, 2025
@mannycarrera4 mannycarrera4 marked this pull request as ready for review May 30, 2025 20:34
@jaclynjessup jaclynjessup requested a review from NicholasBoll June 2, 2025 16:09
@@ -36,4 +36,3 @@ export {
getCursor,
isCursor,
} from './lib/useCursorListModel';
export {VirtualItem, ScrollToOffsetOptions} from './lib/react-virtual';
Copy link
Contributor Author

Choose a reason for hiding this comment

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

no one is using this

Copy link
Contributor

@RayRedGoose RayRedGoose left a comment

Choose a reason for hiding this comment

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

👍

@RayRedGoose RayRedGoose added the approved Code has been reviewed and approved (ship it) label Jun 4, 2025
@mannycarrera4 mannycarrera4 added automerge and removed ready for review Code is ready for review labels Jun 4, 2025
@alanbsmith alanbsmith merged commit 20b7745 into Workday:prerelease/minor Jun 4, 2025
20 of 21 checks passed
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in Canvas Kit Jun 4, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
approved Code has been reviewed and approved (ship it) automerge
Projects
Status: ✅ Done
Development

Successfully merging this pull request may close these issues.

3 participants