Fix Next.js Pages Router pathname in AppProvider (#4657)

mui · Feb 4, 2025 · 3a3821f · 3a3821f
1 parent 3eaae57
commit 3a3821f
Show file tree

Hide file tree

Showing 3 changed files with 5 additions and 5 deletions.
diff --git a/packages/toolpad-core/src/nextjs/NextAppProviderPages.tsx b/packages/toolpad-core/src/nextjs/NextAppProviderPages.tsx
@@ -8,7 +8,7 @@ import type { AppProviderProps, Navigate, Router } from '../AppProvider';
  * @ignore - internal component.
  */
 export function NextAppProviderPages(props: AppProviderProps) {
-  const { push, replace, asPath, query } = useRouter();
+  const { push, replace, pathname, query } = useRouter();
 
   const search = React.useMemo(() => {
     const params = new URLSearchParams();
@@ -38,11 +38,11 @@ export function NextAppProviderPages(props: AppProviderProps) {
 
   const routerImpl = React.useMemo<Router>(
     () => ({
-      pathname: asPath,
+      pathname,
       searchParams,
       navigate,
     }),
-    [asPath, navigate, searchParams],
+    [navigate, pathname, searchParams],
   );
 
   return <AppProvider router={routerImpl} {...props} />;

diff --git a/playground/nextjs-pages/next-env.d.ts b/playground/nextjs-pages/next-env.d.ts
@@ -3,4 +3,4 @@
 /// <reference types="next/navigation-types/compat/navigation" />
 
 // NOTE: This file should not be edited
-// see https://nextjs.org/docs/app/building-your-application/configuring/typescript for more information.
+// see https://nextjs.org/docs/app/api-reference/config/typescript for more information.
diff --git a/playground/nextjs/next-env.d.ts b/playground/nextjs/next-env.d.ts
@@ -2,4 +2,4 @@
 /// <reference types="next/image-types/global" />
 
 // NOTE: This file should not be edited
-// see https://nextjs.org/docs/app/building-your-application/configuring/typescript for more information.
+// see https://nextjs.org/docs/app/api-reference/config/typescript for more information.