ob/sprint
1
0
Fork 0
mirror of https://github.com/hex248/sprint.git synced 2026-02-08 02:33:01 +00:00
Code Issues Packages Projects Releases Wiki Activity

only show status remove warning if it is being used

Browse Source
This commit is contained in:
Oliver Bryan
2026-01-11 00:14:06 +00:00
parent 69c8ac7bd0
commit 558d0aa3c8
8 changed files with 118 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

43
packages/frontend/src/components/organisations-dialog.tsx
View File

@@ -52,6 +52,7 @@ function OrganisationsDialog({
const [newStatusColour, setNewStatusColour] = useState(DEFAULT_STATUS_COLOUR);
const [statusError, setStatusError] = useState<string | null>(null);
const [statusToRemove, setStatusToRemove] = useState<string | null>(null);
const [issuesUsingStatus, setIssuesUsingStatus] = useState<number>(0);
const [reassignToStatus, setReassignToStatus] = useState<string>("");
const [confirmDialog, setConfirmDialog] = useState<{
@@ -218,11 +219,34 @@ function OrganisationsDialog({
setStatusError(null);
};
const handleRemoveStatusClick = (status: string) => {
if (Object.keys(statuses).length <= 1) return;
setStatusToRemove(status);
const remaining = Object.keys(statuses).filter((s) => s !== status);
setReassignToStatus(remaining[0] || "");
const handleRemoveStatusClick = async (status: string) => {
if (Object.keys(statuses).length <= 1 || !selectedOrganisation) return;
try {
await issue.statusCount({
organisationId: selectedOrganisation.Organisation.id,
status,
onSuccess: (data) => {
const count = (data as { count?: number }).count ?? 0;
if (count > 0) {
setStatusToRemove(status);
setIssuesUsingStatus(count);
const remaining = Object.keys(statuses).filter((s) => s !== status);
setReassignToStatus(remaining[0] || "");
return;
}
const nextStatuses = Object.keys(statuses).filter((s) => s !== status);
void updateStatuses(
Object.fromEntries(nextStatuses.map((statusKey) => [statusKey, statuses[statusKey]])),
);
},
onError: (error) => {
console.error("error checking status usage:", error);
},
});
} catch (err) {
console.error("error checking status usage:", err);
}
};
const moveStatus = async (status: string, direction: "up" | "down") => {
@@ -276,7 +300,7 @@ function OrganisationsDialog({
)}
</DialogTrigger>
<DialogContent className="sm:max-w-md w-full max-w-[calc(100vw-2rem)]">
<DialogContent className="max-w-lg w-full max-w-[calc(100vw-2rem)]">
<DialogHeader>
<DialogTitle>Organisations</DialogTitle>
</DialogHeader>
@@ -471,7 +495,7 @@ function OrganisationsDialog({
Object.keys(statuses).length <= 1
}
onSelect={() =>
handleRemoveStatusClick(status)
void handleRemoveStatusClick(status)
}
className="hover:bg-destructive/10"
>
@@ -587,7 +611,7 @@ function OrganisationsDialog({
}
}}
>
<DialogContent className="sm:max-w-lg">
<DialogContent className="w-md">
<DialogHeader>
<DialogTitle>Remove Status</DialogTitle>
</DialogHeader>
@@ -596,7 +620,8 @@ function OrganisationsDialog({
{statusToRemove ? (
<StatusTag status={statusToRemove} colour={statuses[statusToRemove]} />
) : null}{" "}
status? Which status would you like issues with this status to be set to?
status? <span className="font-700 text-foreground">{issuesUsingStatus}</span>{" "}
issues are using it. Which status would you like these issues to use instead?
</p>
<Select value={reassignToStatus} onValueChange={setReassignToStatus}>
<SelectTrigger className="w-min">

3
packages/frontend/src/components/status-tag.tsx
View File

@@ -1,3 +1,4 @@
import { DEFAULT_STATUS_COLOUR } from "@issue/shared";
import { cn } from "@/lib/utils";
const DARK_TEXT_COLOUR = "#0a0a0a";
@@ -14,7 +15,7 @@ const isLight = (hex: string): boolean => {
export default function StatusTag({
status,
colour,
colour = DEFAULT_STATUS_COLOUR,
className,
}: {
status: string;