Revert "hide admin nav links unless user has admin role"
This reverts commit 92a256bbe0.
This commit is contained in:
@@ -48,7 +48,6 @@ export default function Header() {
|
|||||||
const [searchFocused, setSearchFocused] = useState(false);
|
const [searchFocused, setSearchFocused] = useState(false);
|
||||||
const [currentUserId, setCurrentUserId] = useState<string | null>(null);
|
const [currentUserId, setCurrentUserId] = useState<string | null>(null);
|
||||||
const [currentUserEmail, setCurrentUserEmail] = useState<string | null>(null);
|
const [currentUserEmail, setCurrentUserEmail] = useState<string | null>(null);
|
||||||
const [isAdmin, setIsAdmin] = useState(false);
|
|
||||||
const [subEmail, setSubEmail] = useState("");
|
const [subEmail, setSubEmail] = useState("");
|
||||||
const [subStatus, setSubStatus] = useState<"idle" | "loading" | "success" | "error">("idle");
|
const [subStatus, setSubStatus] = useState<"idle" | "loading" | "success" | "error">("idle");
|
||||||
const [subMessage, setSubMessage] = useState("");
|
const [subMessage, setSubMessage] = useState("");
|
||||||
@@ -66,28 +65,14 @@ export default function Header() {
|
|||||||
|
|
||||||
useEffect(() => {
|
useEffect(() => {
|
||||||
const supabase = createClient();
|
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 } }) => {
|
supabase.auth.getUser().then(({ data: { user } }) => {
|
||||||
setCurrentUserId(user?.id || null);
|
setCurrentUserId(user?.id || null);
|
||||||
setCurrentUserEmail(user?.email || 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);
|
setCurrentUserId(session?.user?.id || null);
|
||||||
setCurrentUserEmail(session?.user?.email || null);
|
setCurrentUserEmail(session?.user?.email || null);
|
||||||
refreshAdmin(session?.user?.id || null);
|
|
||||||
});
|
});
|
||||||
return () => subscription.unsubscribe();
|
return () => subscription.unsubscribe();
|
||||||
}, []);
|
}, []);
|
||||||
@@ -252,8 +237,6 @@ export default function Header() {
|
|||||||
My Profile
|
My Profile
|
||||||
</Link>
|
</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">
|
<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
|
WP Import
|
||||||
</Link>
|
</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">
|
<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
|
Dashboard
|
||||||
</Link>
|
</Link>
|
||||||
</>
|
|
||||||
)}
|
|
||||||
{/* Inline Email Subscription Form */}
|
{/* Inline Email Subscription Form */}
|
||||||
{subStatus === "success" ? (
|
{subStatus === "success" ? (
|
||||||
<span className="text-[#3b82f6] text-xs font-body font-bold whitespace-nowrap">{subMessage}</span>
|
<span className="text-[#3b82f6] text-xs font-body font-bold whitespace-nowrap">{subMessage}</span>
|
||||||
@@ -467,7 +448,6 @@ export default function Header() {
|
|||||||
Account
|
Account
|
||||||
</Link>
|
</Link>
|
||||||
{/* WP Import mobile link */}
|
{/* WP Import mobile link */}
|
||||||
{isAdmin && (
|
|
||||||
<Link
|
<Link
|
||||||
href="/admin/import"
|
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"
|
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>
|
</svg>
|
||||||
WP Import
|
WP Import
|
||||||
</Link>
|
</Link>
|
||||||
)}
|
|
||||||
</div>
|
</div>
|
||||||
<div className="flex flex-col gap-0 border-t border-[#252525] mt-1">
|
<div className="flex flex-col gap-0 border-t border-[#252525] mt-1">
|
||||||
{categoryLinks?.map((link) => (
|
{categoryLinks?.map((link) => (
|
||||||
|
|||||||
Reference in New Issue
Block a user