Skip to content

feat:Replace NEXT_PUBLIC_MOCK_MAINTAINER env flag with useUserRole hook #303

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Benjtalkshow merged 3 commits into from
Jun 27, 2026
Merged

feat:Replace NEXT_PUBLIC_MOCK_MAINTAINER env flag with useUserRole hook #303

Changes from all commits
Commits
Show all changes
3 commits
Select commit Hold shift + click to select a range
fd4f4f3
feat:Replace NEXT_PUBLIC_MOCK_MAINTAINER env flag with useUserRole hook
Olajcodes Jun 27, 2026
de5d6ab
feat:Implemented CodeRabbit feedback
Olajcodes Jun 27, 2026
dfd90fd
fix: add trailing newline to bounty-sidebar.tsx
Olajcodes Jun 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
27 changes: 10 additions & 17 deletions components/bounty/bounty-sidebar.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import { useMemo, useState } from "react";
import { RatingModal } from "../rating/rating-modal";
import { useClaimBounty } from "@/hooks/use-bounty-mutations";
import { useUserRole } from "@/hooks/use-user-role";
import { Button } from "@/components/ui/button";
import { Separator } from "@/components/ui/separator";
import type {
Expand All @@ -28,17 +29,8 @@ export function BountySidebar({ bounty }: BountySidebarProps) {
const claimBounty = useClaimBounty();
const [loading, setLoading] = useState(false);

// Mock maintainer check for now - in real app this comes from auth context
const IS_MAINTAINER = process.env.NEXT_PUBLIC_MOCK_MAINTAINER === "true";

if (
typeof window !== "undefined" &&
process.env.NEXT_PUBLIC_MOCK_MAINTAINER === "true"
) {
console.warn(
"DEV: Mock maintainer enabled in components/bounty/bounty-sidebar.tsx — do NOT enable in production",
);
}
const role = useUserRole();
const isSponsor = role === "sponsor";

const createdTimeAgo = useMemo(
() => formatDistanceToNow(new Date(bounty.createdAt), { addSuffix: true }),
Expand Down Expand Up @@ -90,8 +82,8 @@ export function BountySidebar({ bounty }: BountySidebarProps) {
} | null>(null);

const handleMarkCompleted = async () => {
if (!IS_MAINTAINER) {
alert("Only maintainers can mark as completed.");
if (!isSponsor) {
alert("Only sponsors can mark as completed.");
return;
Comment thread
coderabbitai[bot] marked this conversation as resolved.
}
setLoading(true);
Expand Down Expand Up @@ -121,8 +113,8 @@ export function BountySidebar({ bounty }: BountySidebarProps) {
alert("You have already rated this contributor.");
return;
}
if (!IS_MAINTAINER) {
alert("Only maintainers can rate contributors.");
if (!isSponsor) {
alert("Only sponsors can rate contributors.");
return;
}
if (!completed) {
Expand All @@ -142,7 +134,7 @@ export function BountySidebar({ bounty }: BountySidebarProps) {
};

const renderActionButton = () => {
if (bounty.status === "IN_PROGRESS" && IS_MAINTAINER && !completed) {
if (bounty.status === "IN_PROGRESS" && isSponsor && !completed) {
return (
<Button
onClick={handleMarkCompleted}
Expand Down Expand Up @@ -205,7 +197,7 @@ export function BountySidebar({ bounty }: BountySidebarProps) {
{lastRating && reputationGain && (
<div className="p-4 mb-4 rounded bg-green-900/60 text-green-200 border border-green-700">
<div className="mb-1">
{IS_MAINTAINER
{isSponsor
? "You rated the contributor:"
: "Contributor was rated:"}{" "}
<b>{lastRating} / 5</b> stars
Expand Down Expand Up @@ -260,3 +252,4 @@ export function BountySidebar({ bounty }: BountySidebarProps) {
</div>
);
}

Loading