feat: enhance TicketDetailDialog with status change confirmation and dropdown menu for status selection
This commit is contained in:
@@ -1,12 +1,22 @@
|
||||
"use client"
|
||||
|
||||
import { useCallback, useEffect, useRef, useState } from "react"
|
||||
import { MessageSquareTextIcon, PencilIcon, PlusIcon, RefreshCcwIcon, SendIcon, UserRoundIcon } from "lucide-react"
|
||||
import {
|
||||
CheckIcon,
|
||||
ChevronDownIcon,
|
||||
MessageSquareTextIcon,
|
||||
PencilIcon,
|
||||
PlusIcon,
|
||||
RefreshCcwIcon,
|
||||
SendIcon,
|
||||
UserRoundIcon,
|
||||
} from "lucide-react"
|
||||
import { toast } from "sonner"
|
||||
|
||||
import { type CustomerFormSavePayload } from "@/components/customer-form"
|
||||
import { CustomerFormDialog } from "@/components/customer-form-dialog"
|
||||
import { CustomerLinkOrCreateDialog } from "@/components/customer-link-or-create-dialog"
|
||||
import { useConfirm } from "@/components/confirm-provider"
|
||||
import { ContentEditor } from "@/components/content-editor"
|
||||
import { ProjectDialog } from "@/components/project-dialog"
|
||||
import { isRichTextEmpty, SafeRichHTML } from "@/components/safe-rich-html"
|
||||
@@ -19,6 +29,12 @@ import {
|
||||
DialogHeader,
|
||||
DialogTitle,
|
||||
} from "@/components/ui/dialog"
|
||||
import {
|
||||
DropdownMenu,
|
||||
DropdownMenuContent,
|
||||
DropdownMenuItem,
|
||||
DropdownMenuTrigger,
|
||||
} from "@/components/ui/dropdown-menu"
|
||||
import { Separator } from "@/components/ui/separator"
|
||||
import { saveCustomerProfile } from "@/lib/api/customer"
|
||||
import {
|
||||
@@ -35,7 +51,7 @@ import { useI18n } from "@/i18n/provider"
|
||||
import { cn, formatDateTime } from "@/lib/utils"
|
||||
import { EditDialog } from "./edit"
|
||||
import { TicketAssignDialog } from "./ticket-assign-dialog"
|
||||
import { TicketStatusBadge } from "./ticket-status-badge"
|
||||
import { getTicketStatusMeta, TicketStatusBadge } from "./ticket-status-badge"
|
||||
|
||||
type TicketDetailDialogProps = {
|
||||
ticketId: number | null
|
||||
@@ -54,6 +70,10 @@ function getStatusOptions(t: TFunction): Array<{ value: TicketStatus; label: str
|
||||
]
|
||||
}
|
||||
|
||||
function getStatusLabel(status: TicketStatus, t: TFunction) {
|
||||
return getStatusOptions(t).find((option) => option.value === status)?.label ?? status
|
||||
}
|
||||
|
||||
function sourceLabel(source: string, t: TFunction) {
|
||||
switch (source) {
|
||||
case "manual":
|
||||
@@ -83,6 +103,7 @@ export function TicketDetailDialog({
|
||||
onChanged,
|
||||
}: TicketDetailDialogProps) {
|
||||
const t = useI18n()
|
||||
const confirm = useConfirm()
|
||||
const [detail, setDetail] = useState<TicketDetail | null>(null)
|
||||
const [loading, setLoading] = useState(false)
|
||||
const [statusSaving, setStatusSaving] = useState<TicketStatus | null>(null)
|
||||
@@ -158,6 +179,17 @@ export function TicketDetailDialog({
|
||||
if (!detail || detail.ticket.status === status) {
|
||||
return
|
||||
}
|
||||
const confirmed = await confirm({
|
||||
title: t("ticket.statusChangeConfirmTitle"),
|
||||
description: t("ticket.statusChangeConfirmDescription", {
|
||||
from: getStatusLabel(detail.ticket.status, t),
|
||||
to: getStatusLabel(status, t),
|
||||
}),
|
||||
confirmText: t("ticket.statusChangeConfirm"),
|
||||
})
|
||||
if (!confirmed) {
|
||||
return
|
||||
}
|
||||
const activeTicketId = detail.ticket.id
|
||||
const activeDialogSeq = dialogSeqRef.current
|
||||
setStatusSaving(status)
|
||||
@@ -363,20 +395,42 @@ export function TicketDetailDialog({
|
||||
|
||||
<section className="grid gap-4 md:grid-cols-[minmax(0,1fr)_220px]">
|
||||
<div className="space-y-2">
|
||||
<div className="text-sm font-medium text-muted-foreground">{t("ticket.columnStatus")}</div>
|
||||
<div className="flex flex-wrap gap-2">
|
||||
{statusOptions.map((option) => (
|
||||
<Button
|
||||
key={option.value}
|
||||
type="button"
|
||||
size="sm"
|
||||
variant={ticket.status === option.value ? "default" : "outline"}
|
||||
disabled={!!statusSaving}
|
||||
onClick={() => void handleStatusChange(option.value)}
|
||||
<div className="text-sm font-medium text-muted-foreground">{t("ticket.currentStatus")}</div>
|
||||
<div className="flex flex-wrap items-center gap-2">
|
||||
<DropdownMenu>
|
||||
<DropdownMenuTrigger
|
||||
render={
|
||||
<Button
|
||||
type="button"
|
||||
variant="outline"
|
||||
disabled={!!statusSaving}
|
||||
className={cn(
|
||||
"h-9 min-w-36 justify-between gap-2 border px-3 font-medium",
|
||||
getTicketStatusMeta(ticket.status)?.className,
|
||||
)}
|
||||
/>
|
||||
}
|
||||
>
|
||||
{statusSaving === option.value ? t("ticket.updating") : option.label}
|
||||
</Button>
|
||||
))}
|
||||
<span>{statusSaving ? t("ticket.updating") : getStatusLabel(ticket.status, t)}</span>
|
||||
<ChevronDownIcon className="size-4 opacity-70" />
|
||||
</DropdownMenuTrigger>
|
||||
<DropdownMenuContent align="start" className="w-44 min-w-44">
|
||||
{statusOptions.map((option) => {
|
||||
const selected = ticket.status === option.value
|
||||
return (
|
||||
<DropdownMenuItem
|
||||
key={option.value}
|
||||
disabled={!!statusSaving || selected}
|
||||
onClick={() => void handleStatusChange(option.value)}
|
||||
className="justify-between"
|
||||
>
|
||||
<span>{option.label}</span>
|
||||
{selected ? <CheckIcon className="size-4 text-primary" /> : null}
|
||||
</DropdownMenuItem>
|
||||
)
|
||||
})}
|
||||
</DropdownMenuContent>
|
||||
</DropdownMenu>
|
||||
</div>
|
||||
</div>
|
||||
<div className="space-y-2">
|
||||
|
||||
@@ -14,9 +14,13 @@ export function ticketStatusLabel(status: string) {
|
||||
return status
|
||||
}
|
||||
|
||||
export function getTicketStatusMeta(status: string) {
|
||||
return statusMap[status as TicketStatus]
|
||||
}
|
||||
|
||||
export function TicketStatusBadge({ status }: { status: string }) {
|
||||
const t = useI18n()
|
||||
const option = statusMap[status as TicketStatus]
|
||||
const option = getTicketStatusMeta(status)
|
||||
|
||||
return (
|
||||
<Badge variant="outline" className={option?.className ?? "border-border bg-muted text-muted-foreground"}>
|
||||
|
||||
@@ -1416,6 +1416,7 @@
|
||||
"loadingRows": "Loading tickets...",
|
||||
"emptyRows": "No tickets yet",
|
||||
"detail": "Details",
|
||||
"currentStatus": "Current Status",
|
||||
"statusPending": "Pending",
|
||||
"statusInProgress": "In Progress",
|
||||
"statusDone": "Resolved",
|
||||
@@ -1446,6 +1447,9 @@
|
||||
"loadDetailFailed": "Could not load ticket details.",
|
||||
"statusUpdated": "Ticket status updated.",
|
||||
"statusUpdateFailed": "Could not update ticket status.",
|
||||
"statusChangeConfirmTitle": "Confirm Status Change",
|
||||
"statusChangeConfirmDescription": "Change ticket status from \"{from}\" to \"{to}\"?",
|
||||
"statusChangeConfirm": "Confirm Change",
|
||||
"updating": "Updating...",
|
||||
"progressRequired": "Enter an update.",
|
||||
"progressRecorded": "Update added.",
|
||||
|
||||
@@ -1416,6 +1416,7 @@
|
||||
"loadingRows": "正在加载工单...",
|
||||
"emptyRows": "暂无工单",
|
||||
"detail": "详情",
|
||||
"currentStatus": "当前状态",
|
||||
"statusPending": "待处理",
|
||||
"statusInProgress": "处理中",
|
||||
"statusDone": "已处理",
|
||||
@@ -1446,6 +1447,9 @@
|
||||
"loadDetailFailed": "加载工单详情失败",
|
||||
"statusUpdated": "工单状态已更新",
|
||||
"statusUpdateFailed": "更新工单状态失败",
|
||||
"statusChangeConfirmTitle": "确认修改工单状态",
|
||||
"statusChangeConfirmDescription": "确定要将工单状态从「{from}」修改为「{to}」吗?",
|
||||
"statusChangeConfirm": "确认修改",
|
||||
"updating": "更新中...",
|
||||
"progressRequired": "请填写处理进展",
|
||||
"progressRecorded": "处理进展已记录",
|
||||
|
||||
Reference in New Issue
Block a user