Fix: Upgrade project permission bug

This commit is contained in:
Daniel Hougaard
2024-04-22 21:08:09 +02:00
parent be8232dc93
commit 22a3c46902
@@ -25,23 +25,24 @@ export const UpgradeProjectAlert = ({
project, project,
transparent transparent
}: UpgradeProjectAlertProps): JSX.Element | null => { }: UpgradeProjectAlertProps): JSX.Element | null => {
const router = useRouter(); const router = useRouter();
const { membership } = useProjectPermission(); const { hasProjectRole } = useProjectPermission();
const upgradeProject = useUpgradeProject(); const upgradeProject = useUpgradeProject();
const [currentStatus, setCurrentStatus] = useState<string | null>(null); const [currentStatus, setCurrentStatus] = useState<string | null>(null);
const [isUpgrading, setIsUpgrading] = useState(false); const [isUpgrading, setIsUpgrading] = useState(false);
const isProjectAdmin = hasProjectRole("admin");
const { const {
data: projectStatus, data: projectStatus,
isLoading: statusIsLoading, isLoading: statusIsLoading,
refetch: manualProjectStatusRefetch refetch: manualProjectStatusRefetch
} = useGetUpgradeProjectStatus({ } = useGetUpgradeProjectStatus({
projectId: project.id, projectId: project.id,
enabled: membership.role === "admin" && project.version === ProjectVersion.V1, enabled: isProjectAdmin && project.version === ProjectVersion.V1,
refetchInterval: 5_000, refetchInterval: 5_000,
onSuccess: (data) => { onSuccess: (data) => {
if (membership.role !== "admin") { if (!isProjectAdmin) {
return; return;
} }
@@ -101,7 +102,7 @@ export const UpgradeProjectAlert = ({
variant="solid" variant="solid"
size="md" size="md"
isLoading={isLoading} isLoading={isLoading}
isDisabled={isLoading || membership.role !== "admin"} isDisabled={isLoading || !isProjectAdmin}
onClick={onUpgradeProject} onClick={onUpgradeProject}
> >
Upgrade Upgrade
@@ -113,13 +114,13 @@ export const UpgradeProjectAlert = ({
<div <div
className={twMerge( className={twMerge(
"mt-4 flex w-full flex-row items-center rounded-md border border-primary-600/70 bg-primary/[.07] p-4 text-base text-white", "mt-4 flex w-full flex-row items-center rounded-md border border-primary-600/70 bg-primary/[.07] p-4 text-base text-white",
membership.role !== "admin" && "opacity-80" !isProjectAdmin && "opacity-80"
)} )}
> >
<FontAwesomeIcon icon={faWarning} className="pr-6 text-6xl text-white/80" /> <FontAwesomeIcon icon={faWarning} className="pr-6 text-6xl text-white/80" />
<div className="flex w-full flex-col text-sm"> <div className="flex w-full flex-col text-sm">
<span className="mb-2 text-lg font-semibold">Upgrade your project</span> <span className="mb-2 text-lg font-semibold">Upgrade your project</span>
{membership.role === "admin" ? ( {isProjectAdmin ? (
<> <>
<p> <p>
Upgrade your project version to continue receiving the latest improvements and Upgrade your project version to continue receiving the latest improvements and
@@ -150,12 +151,12 @@ export const UpgradeProjectAlert = ({
</div> </div>
<div className="my-2"> <div className="my-2">
<Tooltip <Tooltip
className={twMerge(membership.role === "admin" && "hidden")} className={twMerge(isProjectAdmin && "hidden")}
content="You need to be an admin to upgrade the project." content="You need to be an admin to upgrade the project."
> >
<Button <Button
isLoading={isLoading} isLoading={isLoading}
isDisabled={isLoading || membership.role !== "admin"} isDisabled={isLoading || !isProjectAdmin}
onClick={onUpgradeProject} onClick={onUpgradeProject}
> >
Upgrade Upgrade