Conversation
🦋 Changeset detectedLatest commit: 11391cc The changes in this PR will be included in the next version bump. This PR includes changesets to release 13 packages
Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
| }) { | ||
| const pathname = usePathname(); | ||
|
|
||
| if (pathname === "/") { |
There was a problem hiding this comment.
I avoided using Next.js Route Groups since there is likely to be some differences with Tanstack we'd need to revert/change any way.
| import { SelectedENSNodeProvider } from "@/components/providers/selected-ensnode-provider"; | ||
| import { SidebarInset, SidebarProvider } from "@/components/ui/sidebar"; | ||
|
|
||
| export function LayoutWrapper({ |
There was a problem hiding this comment.
I like the idea of LayoutWrapper component. We could use it explicitly for regular views, while skipping it for mocked views. This way, the mock views could run without a need for a valid ENSNode connection.
lightwalker-eth
left a comment
lightwalker-eth
left a comment
There was a problem hiding this comment.
@notrab I like the strategy you took here. Thank you. Looks nice 👍
3 participants
This handles the root
/route for ENSAdmin.