localadministrator/avbeat-com
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Revert "hide admin nav links unless user has admin role"

Browse Source 
This reverts commit 92a256bbe0.
This commit is contained in:
Ryan Salazar
2026-05-07 22:50:35 +00:00
parent 92a256bbe0
commit 0d51dade6e
1 changed files with 53 additions and 74 deletions
Download Patch File Download Diff File Expand all files Collapse all files

25
src/components/Header.tsx
View File

@@ -48,7 +48,6 @@ export default function Header() {
const [searchFocused, setSearchFocused] = useState(false);
const [currentUserId, setCurrentUserId] = useState<string | null>(null);
const [currentUserEmail, setCurrentUserEmail] = useState<string | null>(null);
const [isAdmin, setIsAdmin] = useState(false);
const [subEmail, setSubEmail] = useState("");
const [subStatus, setSubStatus] = useState<"idle" | "loading" | "success" | "error">("idle");
const [subMessage, setSubMessage] = useState("");
@@ -66,28 +65,14 @@ export default function Header() {
useEffect(() => {
const supabase = createClient();
const ADMIN_ROLES = new Set(["administrator", "admin", "super_admin"]);
const refreshAdmin = async (userId: string | null) => {
if (!userId) { setIsAdmin(false); return; }
const { data } = await supabase
.from("user_profiles")
.select("role")
.eq("id", userId)
.maybeSingle();
setIsAdmin(!!data?.role && ADMIN_ROLES.has(data.role));
};
supabase.auth.getUser().then(({ data: { user } }) => {
setCurrentUserId(user?.id || null);
setCurrentUserEmail(user?.email || null);
refreshAdmin(user?.id || null);
});
const { data: { subscription } } = supabase.auth.onAuthStateChange((_event, session) => {
const supabase2 = createClient();
const { data: { subscription } } = supabase2.auth.onAuthStateChange((_event, session) => {
setCurrentUserId(session?.user?.id || null);
setCurrentUserEmail(session?.user?.email || null);
refreshAdmin(session?.user?.id || null);
});
return () => subscription.unsubscribe();
}, []);
@@ -252,8 +237,6 @@ export default function Header() {
My Profile
</Link>
)}
{isAdmin && (
<>
<Link href="/admin/import" className="text-[#999] hover:text-[#3b82f6] focus:text-[#3b82f6] focus:outline-none focus-visible:ring-1 focus-visible:ring-[#3b82f6] transition-colors text-xs font-body font-bold uppercase tracking-wider">
WP Import
</Link>
@@ -296,8 +279,6 @@ export default function Header() {
<Link href="/admin" className="text-[#3b82f6] hover:text-blue-300 focus:text-blue-300 focus:outline-none focus-visible:ring-1 focus-visible:ring-[#3b82f6] transition-colors text-xs font-body font-bold uppercase tracking-wider">
Dashboard
</Link>
</>
)}
{/* Inline Email Subscription Form */}
{subStatus === "success" ? (
<span className="text-[#3b82f6] text-xs font-body font-bold whitespace-nowrap">{subMessage}</span>
@@ -467,7 +448,6 @@ export default function Header() {
Account
</Link>
{/* WP Import mobile link */}
{isAdmin && (
<Link
href="/admin/import"
className="px-2 py-2.5 text-sm font-body font-bold uppercase tracking-wide text-[#cccccc] hover:text-[#3b82f6] hover:bg-[#1a1a1a] border-b border-[#222] transition-colors focus:outline-none focus-visible:ring-1 focus-visible:ring-inset focus-visible:ring-[#3b82f6] flex items-center gap-2"
@@ -477,7 +457,6 @@ export default function Header() {
</svg>
WP Import
</Link>
)}
</div>
<div className="flex flex-col gap-0 border-t border-[#252525] mt-1">
{categoryLinks?.map((link) => (