Skip to content

[DO NOT REVIEW]: Upgrade to ReactRouter v6 #2507

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

Closed
wants to merge 5 commits into from
Closed

[DO NOT REVIEW]: Upgrade to ReactRouter v6 #2507

wants to merge 5 commits into from

Conversation

jandrade
Copy link
Member

Summary:

This is a WIP PR to upgrade to React Router v6. This PR is not meant for review,
but rather to enable WB to work with the new CSS View Transition API.

Issue: "none"

Test plan:

Verify that the ReactRouter-related examples still work, for example:

/?path=/story/packages-button--with-router

@jandrade jandrade self-assigned this Mar 18, 2025
Copy link

changeset-bot bot commented Mar 18, 2025

🦋 Changeset detected

Latest commit: 17265b4

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 17 packages
Name Type
@khanacademy/wonder-blocks-testing-core Major
@khanacademy/wonder-blocks-icon-button Major
@khanacademy/wonder-blocks-clickable Major
@khanacademy/wonder-blocks-button Major
@khanacademy/wonder-blocks-link Major
@khanacademy/wonder-blocks-testing Patch
@khanacademy/wonder-blocks-banner Patch
@khanacademy/wonder-blocks-modal Patch
@khanacademy/wonder-blocks-popover Patch
@khanacademy/wonder-blocks-search-field Patch
@khanacademy/wonder-blocks-accordion Patch
@khanacademy/wonder-blocks-cell Patch
@khanacademy/wonder-blocks-dropdown Patch
@khanacademy/wonder-blocks-form Patch
@khanacademy/wonder-blocks-pill Patch
@khanacademy/wonder-blocks-tooltip Patch
@khanacademy/wonder-blocks-birthday-picker Patch

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

@khan-actions-bot khan-actions-bot requested a review from a team March 18, 2025 22:14
@khan-actions-bot
Copy link
Contributor

khan-actions-bot commented Mar 18, 2025

Gerald

Required Reviewers
  • @Khan/wonder-blocks for changes to package.json, pnpm-lock.yaml, pnpm-workspace.yaml, .changeset/green-lemons-sing.md, .storybook/preview.tsx, __docs__/wonder-blocks-button/button.stories.tsx, __docs__/wonder-blocks-button/navigation-callbacks.mdx, __docs__/wonder-blocks-button/navigation-callbacks.stories.tsx, __docs__/wonder-blocks-cell/detail-cell.stories.tsx, __docs__/wonder-blocks-clickable/clickable.stories.tsx, __docs__/wonder-blocks-icon-button/icon-button.stories.tsx, __docs__/wonder-blocks-link/link.stories.tsx, __docs__/wonder-blocks-testing/exports.harness-adapters.mdx, packages/wonder-blocks-button/src/components/button-core.tsx, packages/wonder-blocks-button/src/components/button.tsx, packages/wonder-blocks-clickable/src/components/clickable-behavior.tsx, packages/wonder-blocks-clickable/src/components/clickable.tsx, packages/wonder-blocks-clickable/src/util/get-clickable-behavior.tsx, packages/wonder-blocks-icon-button/src/components/icon-button-core.tsx, packages/wonder-blocks-link/src/components/link-core.tsx, packages/wonder-blocks-link/src/components/link.tsx, packages/wonder-blocks-button/src/components/__tests__/button.test.tsx, packages/wonder-blocks-clickable/src/components/__tests__/clickable-behavior.test.tsx, packages/wonder-blocks-clickable/src/components/__tests__/clickable.test.tsx, packages/wonder-blocks-clickable/src/util/__tests__/get-clickable-behavior.test.tsx, packages/wonder-blocks-icon-button/src/components/__tests__/icon-button.test.tsx, packages/wonder-blocks-link/src/components/__tests__/link.test.tsx, packages/wonder-blocks-testing-core/src/harness/__tests__/make-test-harness.test.tsx, packages/wonder-blocks-testing-core/src/harness/adapters/adapters.ts, packages/wonder-blocks-testing-core/src/harness/adapters/router.tsx, packages/wonder-blocks-testing-core/src/harness/adapters/__tests__/router.test.tsx

Don't want to be involved in this pull request? Comment #removeme and we won't notify you of further changes.

Copy link
Contributor

github-actions bot commented Mar 18, 2025

Size Change: -152 B (-0.15%)

Total Size: 98.7 kB

Filename Size Change
packages/wonder-blocks-button/dist/es/index.js 4.3 kB -40 B (-0.92%)
packages/wonder-blocks-clickable/dist/es/index.js 3.09 kB +22 B (+0.72%)
packages/wonder-blocks-icon-button/dist/es/index.js 3.14 kB -33 B (-1.04%)
packages/wonder-blocks-link/dist/es/index.js 1.97 kB -65 B (-3.19%)
packages/wonder-blocks-testing-core/dist/es/index.js 3.88 kB -36 B (-0.92%)
ℹ️ View Unchanged
Filename Size
packages/wonder-blocks-accordion/dist/es/index.js 3.54 kB
packages/wonder-blocks-banner/dist/es/index.js 1.55 kB
packages/wonder-blocks-birthday-picker/dist/es/index.js 1.88 kB
packages/wonder-blocks-breadcrumbs/dist/es/index.js 886 B
packages/wonder-blocks-cell/dist/es/index.js 2.35 kB
packages/wonder-blocks-core/dist/es/index.js 2.85 kB
packages/wonder-blocks-data/dist/es/index.js 6.25 kB
packages/wonder-blocks-dropdown/dist/es/index.js 19.5 kB
packages/wonder-blocks-form/dist/es/index.js 6.04 kB
packages/wonder-blocks-grid/dist/es/index.js 1.36 kB
packages/wonder-blocks-icon/dist/es/index.js 873 B
packages/wonder-blocks-labeled-field/dist/es/index.js 1.26 kB
packages/wonder-blocks-layout/dist/es/index.js 1.82 kB
packages/wonder-blocks-modal/dist/es/index.js 5.5 kB
packages/wonder-blocks-pill/dist/es/index.js 1.49 kB
packages/wonder-blocks-popover/dist/es/index.js 4.92 kB
packages/wonder-blocks-progress-spinner/dist/es/index.js 1.52 kB
packages/wonder-blocks-search-field/dist/es/index.js 1.32 kB
packages/wonder-blocks-switch/dist/es/index.js 2.02 kB
packages/wonder-blocks-testing/dist/es/index.js 1.07 kB
packages/wonder-blocks-theming/dist/es/index.js 679 B
packages/wonder-blocks-timing/dist/es/index.js 1.79 kB
packages/wonder-blocks-tokens/dist/es/index.js 2.7 kB
packages/wonder-blocks-toolbar/dist/es/index.js 923 B
packages/wonder-blocks-tooltip/dist/es/index.js 7.01 kB
packages/wonder-blocks-typography/dist/es/index.js 1.23 kB

compressed-size-action

Copy link
Contributor

github-actions bot commented Mar 19, 2025

npm Snapshot: Published

🎉 Good news!! We've packaged up the latest commit from this PR (aff5037) and published all packages with changesets to npm.

You can install the packages in webapp by running:

./services/static/dev/tools/deploy_wonder_blocks.js --tag="PR2507"

Packages can also be installed manually by running:

pnpm add @khanacademy/wonder-blocks-<package-name>@PR2507

Copy link
Contributor

A new build was pushed to Chromatic! 🚀

https://5e1bf4b385e3fb0020b7073c-eotalzbeoj.chromatic.com/

Chromatic results:

Metric Total
Captured snapshots 388
Tests with visual changes 0
Total stories 574
Inherited (not captured) snapshots [TurboSnap] 0
Tests on the build 388

@jandrade jandrade marked this pull request as draft April 2, 2025 15:04
@jandrade jandrade removed the request for review from a team April 10, 2025 14:52
@jandrade
Copy link
Member Author

No longer needed

@jandrade jandrade closed this Apr 15, 2025
@jandrade jandrade deleted the rr630 branch April 15, 2025 15:38
Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 0.00%. Comparing base (1d7be37) to head (17265b4).
Report is 55 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@     Coverage Diff      @@
##   main   #2507   +/-   ##
============================
============================

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1d7be37...17265b4. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

2 participants