fix(deps): update react-router monorepo to v6.30.3 by renovate[bot] · Pull Request #935 · openedx/frontend-app-learning

renovate · 2022-04-21T22:25:38Z

This PR contains the following updates:

Package	Change	Age	Confidence
react-router (source)	`6.15.0` → `6.30.3`
react-router-dom (source)	`6.15.0` → `6.30.3`

Release Notes

remix-run/react-router (react-router)

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)
Updated dependencies:
- @remix-run/router@1.21.0

`v6.27.0`

Compare Source

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 types for RouteObject within PatchRoutesOnNavigationFunction's patch method so it doesn't expect agnostic route objects passed to patch (#11967)
Updated dependencies:
- @remix-run/router@1.20.0

`v6.26.2`

Compare Source

Patch Changes

Updated dependencies:
- @remix-run/router@1.19.2

`v6.26.1`

Compare Source

Patch Changes

Rename unstable_patchRoutesOnMiss to unstable_patchRoutesOnNavigation to match new behavior (#11888)
Updated dependencies:
- @remix-run/router@1.19.1

`v6.26.0`

Compare Source

Minor Changes

Add a new replace(url, init?) alternative to redirect(url, init?) that performs a history.replaceState instead of a history.pushState on client-side navigation redirects (#11811)

Patch Changes

Fix initial hydration behavior when using future.v7_partialHydration along with unstable_patchRoutesOnMiss (#11838)
- During initial hydration, router.state.matches will now include any partial matches so that we can render ancestor HydrateFallback components
Updated dependencies:
- @remix-run/router@1.19.0

`v6.25.1`

Compare Source

No significant changes to this package were made in this release. See the repo CHANGELOG.md for an overview of all changes in v6.25.1.

`v6.25.0`

Compare Source

Minor Changes

Stabilize future.unstable_skipActionErrorRevalidation as future.v7_skipActionErrorRevalidation (#11769)
- When this flag is enabled, actions will not automatically trigger a revalidation if they return/throw a Response with a 4xx/5xx status code
- You may still opt-into revalidation via shouldRevalidate
- This also changes shouldRevalidate's unstable_actionStatus parameter to actionStatus

Patch Changes

Fix regression and properly decode paths inside useMatch so matches/params reflect decoded params (#11789)
Updated dependencies:
- @remix-run/router@1.18.0

`v6.24.1`

Compare Source

Patch Changes

When using future.v7_relativeSplatPath, properly resolve relative paths in splat routes that are children of pathless routes (#11633)
Updated dependencies:
- @remix-run/router@1.17.1

`v6.24.0`

Compare Source

Minor Changes

Add support for Lazy Route Discovery (a.k.a. Fog of War) (#11626)
- RFC: https://redirect.github.com/remix-run/react-router/discussions/11113
- unstable_patchRoutesOnMiss docs: https://reactrouter.com/v6/routers/create-browser-router

Patch Changes

Updated dependencies:
- @remix-run/router@1.17.0

`v6.23.1`

Compare Source

Patch Changes

allow undefined to be resolved with <Await> (#11513)
Updated dependencies:
- @remix-run/router@1.16.1

`v6.23.0`

Compare Source

Minor Changes

Add a new unstable_dataStrategy configuration option (#11098)
- This option allows Data Router applications to take control over the approach for executing route loaders and actions
- The default implementation is today's behavior, to fetch all loaders in parallel, but this option allows users to implement more advanced data flows including Remix single-fetch, middleware/context APIs, automatic loader caching, and more

Patch Changes

Updated dependencies:
- @remix-run/router@1.16.0

`v6.22.3`

Compare Source

Patch Changes

Updated dependencies:
- @remix-run/router@1.15.3

`v6.22.2`

Compare Source

Patch Changes

Updated dependencies:
- @remix-run/router@1.15.2

`v6.22.1`

Compare Source

Patch Changes

Fix encoding/decoding issues with pre-encoded dynamic parameter values (#11199)
Updated dependencies:
- @remix-run/router@1.15.1

`v6.22.0`

Compare Source

Patch Changes

Updated dependencies:
- @remix-run/router@1.15.0

`v6.21.3`

Compare Source

Patch Changes

Remove leftover unstable_ prefix from Blocker/BlockerFunction types (#11187)

`v6.21.2`

Compare Source

Patch Changes

Updated dependencies:
- @remix-run/router@1.14.2

`v6.21.1`

Compare Source

Patch Changes

Fix bug with route.lazy not working correctly on initial SPA load when v7_partialHydration is specified (#11121)
Updated dependencies:
- @remix-run/router@1.14.1

`v6.21.0`

Compare Source

Minor Changes

Add a new future.v7_relativeSplatPath flag to implement a breaking bug fix to relative routing when inside a splat route. (#11087)

This fix was originally added in #10983 and was later reverted in #11078 because it was determined that a large number of existing applications were relying on the buggy behavior (see #11052)

The Bug
The buggy behavior is that without this flag, the default behavior when resolving relative paths is to ignore any splat (*) portion of the current route path.

The Background
This decision was originally made thinking that it would make the concept of nested different sections of your apps in <Routes> easier if relative routing would replace the current splat:

<BrowserRouter>
  <Routes>
    <Route path="/" element={<Home />} />
    <Route path="dashboard/*" element={<Dashboard />} />
  </Routes>
</BrowserRouter>

Any paths like /dashboard, /dashboard/team, /dashboard/projects will match the Dashboard route. The dashboard component itself can then render nested <Routes>:

function Dashboard() {
  return (
    <div>
      <h2>Dashboard</h2>
      <nav>
        <Link to="/">Dashboard Home</Link>
        <Link to="team">Team</Link>
        <Link to="projects">Projects</Link>
      </nav>

      <Routes>
        <Route path="/" element={<DashboardHome />} />
        <Route path="team" element={<DashboardTeam />} />
        <Route path="projects" element={<DashboardProjects />} />
      </Routes>
    </div>
  );
}

Now, all links and route paths are relative to the router above them. This makes code splitting and compartmentalizing your app really easy. You could render the Dashboard as its own independent app, or embed it into your large app without making any changes to it.

The Problem

The problem is that this concept of ignoring part of a path breaks a lot of other assumptions in React Router - namely that "." always means the current location pathname for that route. When we ignore the splat portion, we start getting invalid paths when using ".":

// If we are on URL /dashboard/team, and we want to link to /dashboard/team:
function DashboardTeam() {
  // ❌ This is broken and results in <a href="/dashboard">
  return <Link to=".">A broken link to the Current URL</Link>;

  // ✅ This is fixed but super unintuitive since we're already at /dashboard/team!
  return <Link to="./team">A broken link to the Current URL</Link>;
}

We've also introduced an issue that we can no longer move our DashboardTeam component around our route hierarchy easily - since it behaves differently if we're underneath a non-splat route, such as /dashboard/:widget. Now, our "." links will, properly point to ourself inclusive of the dynamic param value so behavior will break from it's corresponding usage in a /dashboard/* route.

Even worse, consider a nested splat route configuration:

<BrowserRouter>
  <Routes>
    <Route path="dashboard">
      <Route path="*" element={<Dashboard />} />
    </Route>
  </Routes>
</BrowserRouter>

Now, a <Link to="."> and a <Link to=".."> inside the Dashboard component go to the same place! That is definitely not correct!

Another common issue arose in Data Routers (and Remix) where any <Form> should post to it's own route action if you the user doesn't specify a form action:

let router = createBrowserRouter({
  path: "/dashboard",
  children: [
    {
      path: "*",
      action: dashboardAction,
      Component() {
        // ❌ This form is broken!  It throws a 405 error when it submits because
        // it tries to submit to /dashboard (without the splat value) and the parent
        // `/dashboard` route doesn't have an action
        return <Form method="post">...</Form>;
      },
    },
  ],
});

This is just a compounded issue from the above because the default location for a Form to submit to is itself (".") - and if we ignore the splat portion, that now resolves to the parent route.

The Solution
If you are leveraging this behavior, it's recommended to enable the future flag, move your splat to it's own route, and leverage ../ for any links to "sibling" pages:

<BrowserRouter>
  <Routes>
    <Route path="dashboard">
      <Route index path="*" element={<Dashboard />} />
    </Route>
  </Routes>
</BrowserRouter>

function Dashboard() {
  return (
    <div>
      <h2>Dashboard</h2>
      <nav>
        <Link to="..">Dashboard Home</Link>
        <Link to="../team">Team</Link>
        <Link to="../projects">Projects</Link>
      </nav>

      <Routes>
        <Route path="/" element={<DashboardHome />} />
        <Route path="team" element={<DashboardTeam />} />
        <Route path="projects" element={<DashboardProjects />} />
      </Router>
    </div>
  );
}

This way, . means "the full current pathname for my route" in all cases (including static, dynamic, and splat routes) and .. always means "my parents pathname".

Patch Changes

Properly handle falsy error values in ErrorBoundary's (#11071)
Updated dependencies:
- @remix-run/router@1.14.0

`v6.20.1`

Compare Source

Patch Changes

Revert the useResolvedPath fix for splat routes due to a large number of applications that were relying on the buggy behavior (see #11052 (comment)). We plan to re-introduce this fix behind a future flag in the next minor version. (#11078)
Updated dependencies:
- @remix-run/router@1.13.1

`v6.20.0`

Compare Source

Minor Changes

Export the PathParam type from the public API (#10719)

Patch Changes

Fix bug with resolveTo in splat routes (#11045)
- This is a follow up to #10983 to handle the few other code paths using getPathContributingMatches
- This removes the UNSAFE_getPathContributingMatches export from @remix-run/router since we no longer need this in the react-router/react-router-dom layers
Updated dependencies:
- @remix-run/router@1.13.0

`v6.19.0`

Compare Source

Minor Changes

Add unstable_flushSync option to useNavigate/useSumbit/fetcher.load/fetcher.submit to opt-out of React.startTransition and into ReactDOM.flushSync for state updates (#11005)
Remove the unstable_ prefix from the useBlocker hook as it's been in use for enough time that we are confident in the API. We do not plan to remove the prefix from unstable_usePrompt due to differences in how browsers handle window.confirm that prevent React Router from guaranteeing consistent/correct behavior. (#10991)

Patch Changes

Fix useActionData so it returns proper contextual action data and not any action data in the tree (#11023)
Fix bug in useResolvedPath that would cause useResolvedPath(".") in a splat route to lose the splat portion of the URL path. (#10983)
- ⚠️ This fixes a quite long-standing bug specifically for "." paths inside a splat route which incorrectly dropped the splat portion of the URL. If you are relative routing via "." inside a splat route in your application you should double check that your logic is not relying on this buggy behavior and update accordingly.
Updated dependencies:
- @remix-run/router@1.12.0

`v6.18.0`

Compare Source

Patch Changes

Fix the future prop on BrowserRouter, HashRouter and MemoryRouter so that it accepts a Partial<FutureConfig> instead of requiring all flags to be included. (#10962)
Updated dependencies:
- @remix-run/router@1.11.0

`v6.17.0`

Compare Source

Patch Changes

Fix RouterProvider future prop type to be a Partial<FutureConfig> so that not all flags must be specified (#10900)
Updated dependencies:
- @remix-run/router@1.10.0

`v6.16.0`

Compare Source

Minor Changes

In order to move towards stricter TypeScript support in the future, we're aiming to replace current usages of any with unknown on exposed typings for user-provided data. To do this in Remix v2 without introducing breaking changes in React Router v6, we have added generics to a number of shared types. These continue to default to any in React Router and are overridden with unknown in Remix. In React Router v7 we plan to move these to unknown as a breaking change. (#10843)
- Location now accepts a generic for the location.state value
- ActionFunctionArgs/ActionFunction/LoaderFunctionArgs/LoaderFunction now accept a generic for the context parameter (only used in SSR usages via createStaticHandler)
- The return type of useMatches (now exported as UIMatch) accepts generics for match.data and match.handle - both of which were already set to unknown
Move the @private class export ErrorResponse to an UNSAFE_ErrorResponseImpl export since it is an implementation detail and there should be no construction of ErrorResponse instances in userland. This frees us up to export a type ErrorResponse which correlates to an instance of the class via InstanceType. Userland code should only ever be using ErrorResponse as a type and should be type-narrowing via isRouteErrorResponse. (#10811)
Export ShouldRevalidateFunctionArgs interface (#10797)
Removed private/internal APIs only required for the Remix v1 backwards compatibility layer and no longer needed in Remix v2 (_isFetchActionRedirect, _hasFetcherDoneAnything) (#10715)

Patch Changes

Updated dependencies:
- @remix-run/router@1.9.0

remix-run/react-router (react-router-dom)

`v6.30.3`

Compare Source

`v6.30.2`

Compare Source

`v6.30.1`

Compare Source

`v6.30.0`

Compare Source

`v6.29.0`

Compare Source

`v6.28.2`

Compare Source

`v6.28.1`

Compare Source

`v6.28.0`

Compare Source

`v6.27.0`

Compare Source

`v6.26.2`

Compare Source

`v6.26.1`

Compare Source

`v6.26.0`

Compare Source

`v6.25.1`

Compare Source

`v6.25.0`

Compare Source

`v6.24.1`

Compare Source

`v6.24.0`

Compare Source

`v6.23.1`

Compare Source

`v6.23.0`

Compare Source

`v6.22.3`

Compare Source

`v6.22.2`

Compare Source

`v6.22.1`

Compare Source

`v6.22.0`

Compare Source

`v6.21.3`

Compare Source

`v6.21.2`

Compare Source

`v6.21.1`

Compare Source

`v6.21.0`

Compare Source

`v6.20.1`

Compare Source

`v6.20.0`

Compare Source

`v6.19.0`

Compare Source

`v6.18.0`

Compare Source

`v6.17.0`

Compare Source

`v6.16.0`

Compare Source

Configuration

📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about these updates again.

If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

codecov · 2022-04-21T22:31:22Z

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.35%. Comparing base (d880aac) to head (4d1080c).
Report is 1 commits behind head on master.

❗ Current head 4d1080c differs from pull request most recent head 91ca79f

Please upload reports for the commit 91ca79f to get more accurate results.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #935      +/-   ##
==========================================
- Coverage   88.83%   88.35%   -0.49%     
==========================================
  Files         307      291      -16     
  Lines        5268     4938     -330     
  Branches     1338     1243      -95     
==========================================
- Hits         4680     4363     -317     
+ Misses        572      561      -11     
+ Partials       16       14       -2

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

renovate bot force-pushed the renovate/react-router-monorepo branch from 2c34ade to 446f390 Compare April 24, 2022 21:09

renovate bot force-pushed the renovate/react-router-monorepo branch from 446f390 to 13c7e03 Compare May 6, 2022 11:07

renovate bot force-pushed the renovate/react-router-monorepo branch from 13c7e03 to a746056 Compare May 17, 2022 03:15

renovate bot changed the title ~~fix(deps): update react-router monorepo to v5.3.1~~ fix(deps): update react-router monorepo to v5.3.2 May 17, 2022

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from 11945a3 to ba3e57f Compare May 18, 2022 23:11

renovate bot changed the title ~~fix(deps): update react-router monorepo to v5.3.2~~ fix(deps): update react-router monorepo to v5.3.3 May 18, 2022

renovate bot force-pushed the renovate/react-router-monorepo branch from ba3e57f to d22c14c Compare June 3, 2022 20:39

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from afb5b5c to 49b6f7a Compare June 13, 2022 13:40

renovate bot force-pushed the renovate/react-router-monorepo branch from 49b6f7a to 07f2f65 Compare June 20, 2022 03:15

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from 0846afd to eb53d17 Compare July 26, 2022 05:47

renovate bot force-pushed the renovate/react-router-monorepo branch from eb53d17 to 0770af1 Compare August 7, 2022 22:13

renovate bot force-pushed the renovate/react-router-monorepo branch 3 times, most recently from db5e04e to f62948e Compare September 13, 2022 06:07

renovate bot force-pushed the renovate/react-router-monorepo branch from f62948e to 150bf53 Compare September 14, 2022 14:08

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from 5078ba9 to b8991a3 Compare October 2, 2022 21:19

renovate bot changed the title ~~fix(deps): update react-router monorepo to v5.3.3~~ fix(deps): update react-router monorepo to v5.3.4 Oct 2, 2022

renovate bot force-pushed the renovate/react-router-monorepo branch 3 times, most recently from ffb3190 to a32ec08 Compare October 19, 2022 11:45

renovate bot force-pushed the renovate/react-router-monorepo branch 4 times, most recently from ad51516 to 339f65d Compare October 31, 2022 15:55

renovate bot force-pushed the renovate/react-router-monorepo branch from 339f65d to ca39030 Compare November 3, 2022 13:47

renovate bot force-pushed the renovate/react-router-monorepo branch 8 times, most recently from dff3eb5 to 48fe16d Compare December 28, 2022 09:55

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from adb9676 to 2739c6d Compare January 3, 2023 18:27

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from fa5335c to a55c6a0 Compare January 12, 2023 22:21

renovate bot force-pushed the renovate/react-router-monorepo branch 5 times, most recently from 4986adc to 5fb604b Compare January 22, 2023 23:55

renovate bot force-pushed the renovate/react-router-monorepo branch 5 times, most recently from 370c631 to 5ec2761 Compare January 31, 2023 16:09

renovate bot force-pushed the renovate/react-router-monorepo branch from 5ec2761 to 353ddfa Compare February 6, 2023 00:14

renovate bot force-pushed the renovate/react-router-monorepo branch 2 times, most recently from 360ee73 to 1d0be8c Compare February 15, 2023 01:02

renovate bot force-pushed the renovate/react-router-monorepo branch 4 times, most recently from 1d6ca83 to 713cc20 Compare March 1, 2023 02:15

fix(deps): update react-router monorepo to v6.30.3

b23dfc1

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

fix(deps): update react-router monorepo to v6.30.3#935

fix(deps): update react-router monorepo to v6.30.3#935
renovate[bot] wants to merge 1 commit intomasterfrom
renovate/react-router-monorepo

renovate bot commented Apr 21, 2022 •

edited

Loading

Uh oh!

codecov bot commented Apr 21, 2022 •

edited

Loading

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

0 participants

Conversation

renovate bot commented Apr 21, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Release Notes

v6.30.3: v6.30.3

v6.30.2: v6.30.2

v6.30.1: v6.30.1

v6.30.0: v6.30.0

v6.29.0: v6.29.0

v6.28.2: v6.28.2

v6.28.1: v6.28.1

Minor Changes

Patch Changes

Minor Changes

Patch Changes

Patch Changes

Patch Changes

Minor Changes

Patch Changes

Minor Changes

Patch Changes

Patch Changes

Minor Changes

Patch Changes

Patch Changes

Minor Changes

Patch Changes

Patch Changes

Patch Changes

Patch Changes

Patch Changes

Patch Changes

Patch Changes

Patch Changes

Minor Changes

Patch Changes

Patch Changes

Minor Changes

Patch Changes

Minor Changes

Patch Changes

Patch Changes

Patch Changes

Minor Changes

Patch Changes

Configuration

Uh oh!

codecov bot commented Apr 21, 2022 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Codecov Report

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

0 participants

renovate bot commented Apr 21, 2022 •

edited

Loading

`v6.30.3`: v6.30.3

`v6.30.2`: v6.30.2

`v6.30.1`: v6.30.1

`v6.30.0`: v6.30.0

`v6.29.0`: v6.29.0

`v6.28.2`: v6.28.2

`v6.28.1`: v6.28.1

codecov bot commented Apr 21, 2022 •

edited

Loading