From 35d0ca9e12f063ae4c2b58be1b4dd4bd0dacf279 Mon Sep 17 00:00:00 2001 From: javayhu Date: Fri, 5 Sep 2025 00:06:40 +0800 Subject: [PATCH] refactor: rename getSidebarLinks to useSidebarLinks for improved clarity and consistency --- src/components/dashboard/dashboard-sidebar.tsx | 4 ++-- src/config/sidebar-config.tsx | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/src/components/dashboard/dashboard-sidebar.tsx b/src/components/dashboard/dashboard-sidebar.tsx index 5bdae62..7637589 100644 --- a/src/components/dashboard/dashboard-sidebar.tsx +++ b/src/components/dashboard/dashboard-sidebar.tsx @@ -12,7 +12,7 @@ import { SidebarMenuItem, useSidebar, } from '@/components/ui/sidebar'; -import { getSidebarLinks } from '@/config/sidebar-config'; +import { useSidebarLinks } from '@/config/sidebar-config'; import { LocaleLink } from '@/i18n/navigation'; import { authClient } from '@/lib/auth-client'; import { Routes } from '@/routes'; @@ -35,7 +35,7 @@ export function DashboardSidebar({ const { state } = useSidebar(); // console.log('sidebar currentUser:', currentUser); - const sidebarLinks = getSidebarLinks(); + const sidebarLinks = useSidebarLinks(); const filteredSidebarLinks = sidebarLinks.filter((link) => { if (link.authorizeOnly) { return link.authorizeOnly.includes(currentUser?.role || ''); diff --git a/src/config/sidebar-config.tsx b/src/config/sidebar-config.tsx index 847a1e8..174b9df 100644 --- a/src/config/sidebar-config.tsx +++ b/src/config/sidebar-config.tsx @@ -27,7 +27,7 @@ import { websiteConfig } from './website'; * * @returns The sidebar config with translated titles and descriptions */ -export function getSidebarLinks(): NestedMenuItem[] { +export function useSidebarLinks(): NestedMenuItem[] { const t = useTranslations('Dashboard'); // if is demo website, allow user to access admin and user pages, but data is fake