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.

Sign up for GitHub

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

Jump to bottom

chore: bump @remix-run/router from 1.18.0 to 1.21.0 #2712

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

chore: bump @remix-run/router from 1.18.0 to 1.21.0 #2712

wants to merge 1 commit into from

Conversation

dependabot[bot]
Copy link
Contributor

@dependabot dependabot bot commented on behalf of github Jan 10, 2025

Bumps @remix-run/router from 1.18.0 to 1.21.0.

Changelog

Sourced from @​remix-run/router's changelog.

1.21.0

Minor Changes

    • Log deprecation warnings for v7 flags (#11750)
    • Add deprecation warnings to json/defer in favor of returning raw objects
      • These methods will be removed in React Router v7

Patch Changes

  • Update JSDoc URLs for new website structure (add /v6/ segment) (#12141)

1.20.0

Minor Changes

  • Stabilize unstable_patchRoutesOnNavigation (#11973)
    • Add new PatchRoutesOnNavigationFunctionArgs type for convenience (#11967)
  • Stabilize unstable_dataStrategy (#11974)
  • Stabilize the unstable_flushSync option for navigations and fetchers (#11989)
  • Stabilize the unstable_viewTransition option for navigations and the corresponding unstable_useViewTransitionState hook (#11989)

Patch Changes

  • Fix bug when submitting to the current contextual route (parent route with an index child) when an ?index param already exists from a prior submission (#12003)
  • Fix useFormAction bug - when removing ?index param it would not keep other non-Remix index params (#12003)
  • Fix bug with fetchers not persisting preventScrollReset through redirects during concurrent fetches (#11999)
  • Remove internal cache to fix issues with interrupted patchRoutesOnNavigation calls (#12055)
    • We used to cache in-progress calls to patchRoutesOnNavigation internally so that multiple navigations with the same start/end would only execute the function once and use the same promise
    • However, this approach was at odds with patch short circuiting if a navigation was interrupted (and the request.signal aborted) since the first invocation's patch would no-op
    • This cache also made some assumptions as to what a valid cache key might be - and is oblivious to any other application-state changes that may have occurred
    • So, the cache has been removed because in most cases, repeated calls to something like import() for async routes will already be cached automatically - and if not it's easy enough for users to implement this cache in userland
  • Avoid unnecessary console.error on fetcher abort due to back-to-back revalidation calls (#12050)
  • Expose errors thrown from patchRoutesOnNavigation directly to useRouteError instead of wrapping them in a 400 ErrorResponse instance (#12111)
  • Fix types for RouteObject within PatchRoutesOnNavigationFunction's patch method so it doesn't expect agnostic route objects passed to patch (#11967)
  • Fix bugs with partialHydration when hydrating with errors (#12070)
  • Remove internal discoveredRoutes FIFO queue from unstable_patchRoutesOnNavigation (#11977)

1.19.2

Patch Changes

  • Update the unstable_dataStrategy API to allow for more advanced implementations (#11943)
    • Rename unstable_HandlerResult to unstable_DataStrategyResult
    • The return signature has changed from a parallel array of unstable_DataStrategyResult[] (parallel to matches) to a key/value object of routeId => unstable_DataStrategyResult
      • This allows you to more easily decide to opt-into or out-of revalidating data that may not have been revalidated by default (via match.shouldLoad)
      • ⚠️ This is a breaking change if you've currently adopted unstable_dataStrategy
    • Added a new fetcherKey parameter to unstable_dataStrategy to allow differentiation from navigational and fetcher calls
    • You should now return/throw a result from your handlerOverride instead of returning a DataStrategyResult
      • If you are aggregating the results of match.resolve() into a final results object you should not need to think about the DataStrategyResult type

... (truncated)

Commits

Dependabot compatibility score

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

  • @dependabot rebase will rebase this PR
  • @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
  • @dependabot merge will merge this PR after your CI passes on it
  • @dependabot squash and merge will squash and merge this PR after your CI passes on it
  • @dependabot cancel merge will cancel a previously requested merge and block automerging
  • @dependabot reopen will reopen this PR if it is closed
  • @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
  • @dependabot show <dependency name> ignore conditions will show all of the ignore conditions of the specified dependency
  • @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
  • @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

@dependabot
chore: bump @remix-run/router from 1.18.0 to 1.21.0
e9fd38f 
Bumps [@remix-run/router](https://github.com/remix-run/react-router/tree/HEAD/packages/router) from 1.18.0 to 1.21.0.
- [Release notes](https://github.com/remix-run/react-router/releases)
- [Changelog](https://github.com/remix-run/react-router/blob/@remix-run/[email protected]/packages/router/CHANGELOG.md)
- [Commits](https://github.com/remix-run/react-router/commits/@remix-run/[email protected]/packages/router)

---
updated-dependencies:
- dependency-name: "@remix-run/router"
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@dependabot dependabot bot requested review from odinr and a team as code owners January 10, 2025 18:23
@dependabot dependabot bot added javascript Pull requests that update Javascript code ⛓️ dependencies Pull requests that update a dependency file labels Jan 10, 2025
@changeset-bot changeset-bot
Copy link

changeset-bot bot commented Jan 10, 2025

⚠️ No Changeset found

Latest commit: e9fd38f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@github-actions github-actions bot added the 🚧 chore maintaines work, (update deps, workflos ...) label Jan 10, 2025
@github-actions GitHub Actions
Copy link
Contributor

Coverage Report

Status Category Percentage Covered / Total
🔵 Lines 0.37% 1751 / 461610
🔵 Statements 0.37% 1751 / 461610
🔵 Functions 22.9% 205 / 895
🔵 Branches 36.93% 386 / 1045
File CoverageNo changed files found.
Generated in workflow #8319 for commit e9fd38f by the Vitest Coverage Report Action

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

@odinr odinr Awaiting requested review from odinr odinr is a code owner

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
🚧 chore maintaines work, (update deps, workflos ...) ⛓️ dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
0 participants