Bump react-router from 6.30.3 to 7.18.1 in /examples/react_admin/frontend in the npm_and_yarn group across 1 directory - #23
Open
dependabot[bot] wants to merge 1 commit into
Conversation
Bumps the npm_and_yarn group with 1 update in the /examples/react_admin/frontend directory: [react-router](https://github.com/remix-run/react-router/tree/HEAD/packages/react-router). Updates `react-router` from 6.30.3 to 7.18.1 - [Release notes](https://github.com/remix-run/react-router/releases) - [Changelog](https://github.com/remix-run/react-router/blob/react-router@7.18.1/packages/react-router/CHANGELOG.md) - [Commits](https://github.com/remix-run/react-router/commits/react-router@7.18.1/packages/react-router) --- updated-dependencies: - dependency-name: react-router dependency-version: 7.18.1 dependency-type: indirect dependency-group: npm_and_yarn ... Signed-off-by: dependabot[bot] <support@github.com>
There was a problem hiding this comment.
Dependabot review: react-router 6.30.3 -> 7.18.1 (transitive, lockfile-only)
Verdict: safe to merge. No code changes required.
Scope
- Only
examples/react_admin/frontend/package-lock.jsonchanges.package.jsonis untouched. react-router/react-router-domare indirect deps (dependency-type: indirect), pulled in byreact-admin. They are not imported directly anywhere in the example source (grepacross all.ts/.tsx/.js/.jsx/.htmlfound zero references). The app only usesreact-admincomponents (Admin,Resource,List,Datagrid,TextField,Show,SimpleShowLayout,Layout,CheckForApplicationUpdate).
Compatibility (major v6 -> v7)
react-admin@5.15.1,ra-core@5.15.0andra-ui-materialui@5.15.1all declarereact-router: "^6.28.1 || ^7.1.1"andreact-router-dom: "^6.28.1 || ^7.1.1". 7.18.1 satisfies^7.1.1, so this version of react-admin officially supports react-router v7.- react-router v7 peerDeps
react >=18/react-dom >=18are satisfied (project uses 18.3.1). - react-router v7
engines.node >= 20is advisory only (no engine-strict) and CI does not build the frontend, so no enforcement impact. Node here is v22.
Dependency graph delta
- Removed
@remix-run/router@1.23.2(v6 internal router pkg, folded into v7 core). - Added
cookie@1.1.1andset-cookie-parser@2.7.2(v7 runtime deps).cookie@1.1.1is well above the vulnerable<0.7.0range (CVE-2024-47764); both are clean.
Build / checks
npm ci+vite buildboth succeed with the new lockfile (2324 modules bundled OK).- The pre-existing
tsc --noEmiterrors insrc/dataProvider.ts(unimplemented stub methods +PaginationPayload | undefined) and thenpm auditfindings (ajv / brace-expansion / cross-spawn from the eslint toolchain) are identical on the base lockfile — they are NOT introduced by this bump.
CI impact
.github/workflows/rust.ymlexamplesjob forreact_adminruns onlycargo fmt/update/test; there is no npm/node build step. This lockfile change therefore has no effect on CI.
Approving — the upgrade is within react-admin's supported range, the example builds, and there is no direct react-router usage that could break.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bumps the npm_and_yarn group with 1 update in the /examples/react_admin/frontend directory: react-router.
Updates
react-routerfrom 6.30.3 to 7.18.1Release notes
Sourced from react-router's releases.
... (truncated)
Changelog
Sourced from react-router's changelog.
... (truncated)
Commits
afdf85dRelease v7.18.1 (#15253)6fb1e79Release v7.18.0 (#15187)09e6020Optimize route matching internals (#15186)5b57f5fRequest Host derivation + CSRF check simplifications (#15185)ce596e8Validate RSC redirect protocols (#15177)1cebd2achore: format9d22943Use turbo stream for framework hydration errors (#15175)bf63729Consolidate url normalization logic (#15176)4ce8ff7Fix prerendering pathname issue with trailingSlashAwareDataRequests e… (#15173)4f060ddUse ServerRouter nonce when nonce prop is not specified (#15170)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 rebasewill rebase this PR@dependabot recreatewill recreate this PR, overwriting any edits that have been made to it@dependabot show <dependency name> ignore conditionswill show all of the ignore conditions of the specified dependency@dependabot ignore <dependency name> major versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's major version (unless you unignore this specific dependency's major version or upgrade to it yourself)@dependabot ignore <dependency name> minor versionwill close this group update PR and stop Dependabot creating any more for the specific dependency's minor version (unless you unignore this specific dependency's minor version or upgrade to it yourself)@dependabot ignore <dependency name>will close this group update PR and stop Dependabot creating any more for the specific dependency (unless you unignore this specific dependency or upgrade to it yourself)@dependabot unignore <dependency name>will remove all of the ignore conditions of the specified dependency@dependabot unignore <dependency name> <ignore condition>will remove the ignore condition of the specified dependency and ignore conditionsYou can disable automated security fix PRs for this repo from the Security Alerts page.