Skip to content

fix(react-router-devtools): remove console warnings from useRouter#3770

Merged
birkskyum merged 2 commits intomainfrom
fix(react-router-devtools)--remove-console-warnings-from-useRouter
Mar 13, 2025
Merged

fix(react-router-devtools): remove console warnings from useRouter#3770
birkskyum merged 2 commits intomainfrom
fix(react-router-devtools)--remove-console-warnings-from-useRouter

Conversation

@birkskyum
Copy link
Copy Markdown
Member

@birkskyum birkskyum commented Mar 13, 2025

Closes #3723

The hooks rules makes it impossible to have the same conditional logic for the useRouter call as the solid-router-devtools has, so this mutes the warning to avoid flooding the browser console, because the setup here is intentional.

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Mar 13, 2025

View your CI Pipeline Execution ↗ for commit 1573fe5.

Command Status Duration Result
nx affected --targets=test:eslint,test:unit,tes... ✅ Succeeded 3m 22s View ↗
nx run-many --target=build --exclude=examples/*... ✅ Succeeded 16s View ↗

☁️ Nx Cloud last updated this comment at 2025-03-13 21:48:29 UTC

@pkg-pr-new
Copy link
Copy Markdown

pkg-pr-new Bot commented Mar 13, 2025

Open in Stackblitz

More templates

@tanstack/arktype-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/arktype-adapter@3770

@tanstack/create-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/create-router@3770

@tanstack/directive-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/directive-functions-plugin@3770

@tanstack/eslint-plugin-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/eslint-plugin-router@3770

@tanstack/create-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/create-start@3770

@tanstack/history

npm i https://pkg.pr.new/TanStack/router/@tanstack/history@3770

@tanstack/react-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router@3770

@tanstack/react-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-devtools@3770

@tanstack/react-router-with-query

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-router-with-query@3770

@tanstack/react-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start@3770

@tanstack/react-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-client@3770

@tanstack/react-start-config

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-config@3770

@tanstack/react-start-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-plugin@3770

@tanstack/react-start-router-manifest

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-router-manifest@3770

@tanstack/react-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/react-start-server@3770

@tanstack/router-cli

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-cli@3770

@tanstack/router-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-core@3770

@tanstack/router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools@3770

@tanstack/router-devtools-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-devtools-core@3770

@tanstack/router-generator

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-generator@3770

@tanstack/router-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-plugin@3770

@tanstack/router-utils

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-utils@3770

@tanstack/router-vite-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/router-vite-plugin@3770

@tanstack/server-functions-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/server-functions-plugin@3770

@tanstack/solid-router

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router@3770

@tanstack/solid-router-devtools

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-router-devtools@3770

@tanstack/solid-start

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start@3770

@tanstack/solid-start-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-client@3770

@tanstack/solid-start-config

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-config@3770

@tanstack/solid-start-plugin

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-plugin@3770

@tanstack/solid-start-router-manifest

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-router-manifest@3770

@tanstack/solid-start-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/solid-start-server@3770

@tanstack/start

npm i https://pkg.pr.new/TanStack/router/@tanstack/start@3770

@tanstack/start-api-routes

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-api-routes@3770

@tanstack/start-client-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-client-core@3770

@tanstack/start-config

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-config@3770

@tanstack/start-server-core

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-core@3770

@tanstack/start-server-functions-client

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-client@3770

@tanstack/start-server-functions-fetcher

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-fetcher@3770

@tanstack/start-server-functions-handler

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-handler@3770

@tanstack/start-server-functions-server

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-server@3770

@tanstack/start-server-functions-ssr

npm i https://pkg.pr.new/TanStack/router/@tanstack/start-server-functions-ssr@3770

@tanstack/valibot-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/valibot-adapter@3770

@tanstack/virtual-file-routes

npm i https://pkg.pr.new/TanStack/router/@tanstack/virtual-file-routes@3770

@tanstack/zod-adapter

npm i https://pkg.pr.new/TanStack/router/@tanstack/zod-adapter@3770

commit: 1573fe5

@birkskyum birkskyum merged commit 0e37676 into main Mar 13, 2025
@birkskyum birkskyum deleted the fix(react-router-devtools)--remove-console-warnings-from-useRouter branch March 13, 2025 21:49
leesb971204 pushed a commit to leesb971204/router that referenced this pull request Apr 16, 2025
…anStack#3770)

Closes TanStack#3723

The hooks rules makes it impossible to have the same conditional logic
for the useRouter call as the solid-router-devtools has, so this mutes
the warning to avoid flooding the browser console, because the setup
here is intentional.

---------

Co-authored-by: autofix-ci[bot] <114827586+autofix-ci[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Placing Devtools alongside RouterProvider prints multiple warnings

1 participant