songtianlun/prmbr-image-mksaas
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

refactor: rename getSidebarLinks to useSidebarLinks for improved clarity and consistency

Browse Source
This commit is contained in:
javayhu 2025-09-05 00:06:40 +08:00
parent 34baf20b31
commit 35d0ca9e12
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
src/components/dashboard/dashboard-sidebar.tsx
View File

@ -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 || '');

2
src/config/sidebar-config.tsx
View File

@ -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