feat(projects): add project edit dialog with dropdown menu and archive confirmation
This commit is contained in:
@@ -1,12 +1,29 @@
|
|||||||
"use client";
|
"use client";
|
||||||
|
|
||||||
import { useState, useEffect, useCallback } from "react";
|
import { useState, useEffect, useCallback } from "react";
|
||||||
import { Plus, FolderKanban, ExternalLink } from "lucide-react";
|
import { Plus, FolderKanban, ExternalLink, MoreHorizontal, Pencil, Archive } from "lucide-react";
|
||||||
import { Button } from "@/components/ui/button";
|
import { Button } from "@/components/ui/button";
|
||||||
import { Badge } from "@/components/ui/badge";
|
import { Badge } from "@/components/ui/badge";
|
||||||
import { Progress } from "@/components/ui/progress";
|
import { Progress } from "@/components/ui/progress";
|
||||||
import { Card, CardContent, CardHeader, CardTitle } from "@/components/ui/card";
|
import { Card, CardContent, CardHeader, CardTitle } from "@/components/ui/card";
|
||||||
|
import {
|
||||||
|
DropdownMenu,
|
||||||
|
DropdownMenuContent,
|
||||||
|
DropdownMenuItem,
|
||||||
|
DropdownMenuTrigger,
|
||||||
|
} from "@/components/ui/dropdown-menu";
|
||||||
|
import {
|
||||||
|
AlertDialog,
|
||||||
|
AlertDialogAction,
|
||||||
|
AlertDialogCancel,
|
||||||
|
AlertDialogContent,
|
||||||
|
AlertDialogDescription,
|
||||||
|
AlertDialogFooter,
|
||||||
|
AlertDialogHeader,
|
||||||
|
AlertDialogTitle,
|
||||||
|
} from "@/components/ui/alert-dialog";
|
||||||
import { ProjectCreateDialog } from "@/components/projects/project-create-dialog";
|
import { ProjectCreateDialog } from "@/components/projects/project-create-dialog";
|
||||||
|
import { ProjectEditDialog } from "@/components/projects/project-edit-dialog";
|
||||||
import Link from "next/link";
|
import Link from "next/link";
|
||||||
import { toast } from "sonner";
|
import { toast } from "sonner";
|
||||||
|
|
||||||
@@ -37,6 +54,9 @@ export default function ProjectsPage() {
|
|||||||
const [domainId, setDomainId] = useState<string | null>(null);
|
const [domainId, setDomainId] = useState<string | null>(null);
|
||||||
const [domains, setDomains] = useState<{ id: string; name: string }[]>([]);
|
const [domains, setDomains] = useState<{ id: string; name: string }[]>([]);
|
||||||
const [createOpen, setCreateOpen] = useState(false);
|
const [createOpen, setCreateOpen] = useState(false);
|
||||||
|
const [editProject, setEditProject] = useState<Project | null>(null);
|
||||||
|
const [archiveProject, setArchiveProject] = useState<Project | null>(null);
|
||||||
|
const [archiving, setArchiving] = useState(false);
|
||||||
const [loading, setLoading] = useState(true);
|
const [loading, setLoading] = useState(true);
|
||||||
|
|
||||||
// Fetch domains
|
// Fetch domains
|
||||||
@@ -116,22 +136,23 @@ export default function ProjectsPage() {
|
|||||||
) : (
|
) : (
|
||||||
<div className="grid gap-4 sm:grid-cols-2 lg:grid-cols-3">
|
<div className="grid gap-4 sm:grid-cols-2 lg:grid-cols-3">
|
||||||
{projects.map((project) => (
|
{projects.map((project) => (
|
||||||
<Link key={project.id} href={`/projects/${project.id}`}>
|
<div key={project.id} className="relative">
|
||||||
<Card className="h-full cursor-pointer transition-colors hover:bg-accent/50">
|
<Link href={`/projects/${project.id}`}>
|
||||||
<CardHeader className="pb-2">
|
<Card className="h-full cursor-pointer transition-colors hover:bg-accent/50">
|
||||||
<div className="flex items-start justify-between">
|
<CardHeader className="pb-2">
|
||||||
<div className="flex items-center gap-2">
|
<div className="flex items-start justify-between">
|
||||||
{project.color && (
|
<div className="flex items-center gap-2">
|
||||||
<div
|
{project.color && (
|
||||||
className="h-3 w-3 rounded-full shrink-0"
|
<div
|
||||||
style={{ backgroundColor: project.color }}
|
className="h-3 w-3 rounded-full shrink-0"
|
||||||
/>
|
style={{ backgroundColor: project.color }}
|
||||||
)}
|
/>
|
||||||
<CardTitle className="text-base">{project.name}</CardTitle>
|
)}
|
||||||
|
<CardTitle className="text-base">{project.name}</CardTitle>
|
||||||
|
</div>
|
||||||
|
<ExternalLink className="h-4 w-4 text-muted-foreground shrink-0" aria-hidden="true" />
|
||||||
</div>
|
</div>
|
||||||
<ExternalLink className="h-4 w-4 text-muted-foreground shrink-0" aria-hidden="true" />
|
</CardHeader>
|
||||||
</div>
|
|
||||||
</CardHeader>
|
|
||||||
<CardContent>
|
<CardContent>
|
||||||
{project.description && (
|
{project.description && (
|
||||||
<p className="mb-3 text-sm text-muted-foreground line-clamp-2">{project.description}</p>
|
<p className="mb-3 text-sm text-muted-foreground line-clamp-2">{project.description}</p>
|
||||||
@@ -168,7 +189,32 @@ export default function ProjectsPage() {
|
|||||||
)}
|
)}
|
||||||
</CardContent>
|
</CardContent>
|
||||||
</Card>
|
</Card>
|
||||||
</Link>
|
</Card>
|
||||||
|
</Link>
|
||||||
|
<div className="absolute right-2 top-2">
|
||||||
|
<DropdownMenu>
|
||||||
|
<DropdownMenuTrigger asChild>
|
||||||
|
<button
|
||||||
|
className="rounded-md p-1 text-muted-foreground hover:text-foreground hover:bg-accent transition-colors"
|
||||||
|
aria-label={`Options for ${project.name}`}
|
||||||
|
onClick={(e) => e.stopPropagation()}
|
||||||
|
>
|
||||||
|
<MoreHorizontal className="h-4 w-4" />
|
||||||
|
</button>
|
||||||
|
</DropdownMenuTrigger>
|
||||||
|
<DropdownMenuContent align="end">
|
||||||
|
<DropdownMenuItem onClick={(e) => { e.stopPropagation(); setEditProject(project); }}>
|
||||||
|
<Pencil className="mr-2 h-4 w-4" />
|
||||||
|
Edit
|
||||||
|
</DropdownMenuItem>
|
||||||
|
<DropdownMenuItem onClick={(e) => { e.stopPropagation(); setArchiveProject(project); }}>
|
||||||
|
<Archive className="mr-2 h-4 w-4" />
|
||||||
|
Archive
|
||||||
|
</DropdownMenuItem>
|
||||||
|
</DropdownMenuContent>
|
||||||
|
</DropdownMenu>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
))}
|
))}
|
||||||
</div>
|
</div>
|
||||||
)}
|
)}
|
||||||
@@ -179,6 +225,54 @@ export default function ProjectsPage() {
|
|||||||
domainId={domainId || ''}
|
domainId={domainId || ''}
|
||||||
onCreated={fetchProjects}
|
onCreated={fetchProjects}
|
||||||
/>
|
/>
|
||||||
|
|
||||||
|
{editProject && (
|
||||||
|
<ProjectEditDialog
|
||||||
|
open={!!editProject}
|
||||||
|
onOpenChange={(open) => { if (!open) setEditProject(null); }}
|
||||||
|
project={editProject}
|
||||||
|
domainId={domainId || ''}
|
||||||
|
onUpdated={fetchProjects}
|
||||||
|
/>
|
||||||
|
)}
|
||||||
|
|
||||||
|
<AlertDialog open={!!archiveProject} onOpenChange={(open) => { if (!open) setArchiveProject(null); }}>
|
||||||
|
<AlertDialogContent>
|
||||||
|
<AlertDialogHeader>
|
||||||
|
<AlertDialogTitle>Archive Project</AlertDialogTitle>
|
||||||
|
<AlertDialogDescription>
|
||||||
|
Are you sure you want to archive "{archiveProject?.name}"? It will be hidden from the active list.
|
||||||
|
</AlertDialogDescription>
|
||||||
|
</AlertDialogHeader>
|
||||||
|
<AlertDialogFooter>
|
||||||
|
<AlertDialogCancel>Cancel</AlertDialogCancel>
|
||||||
|
<AlertDialogAction
|
||||||
|
disabled={archiving}
|
||||||
|
onClick={async () => {
|
||||||
|
if (!archiveProject || !domainId) return;
|
||||||
|
setArchiving(true);
|
||||||
|
try {
|
||||||
|
const res = await fetch(`/api/domains/${domainId}/projects/${archiveProject.id}`, {
|
||||||
|
method: 'PATCH',
|
||||||
|
headers: { 'Content-Type': 'application/json' },
|
||||||
|
body: JSON.stringify({ status: 'archived' }),
|
||||||
|
});
|
||||||
|
if (!res.ok) throw new Error('Failed to archive');
|
||||||
|
toast.success('Project archived');
|
||||||
|
setArchiveProject(null);
|
||||||
|
fetchProjects();
|
||||||
|
} catch {
|
||||||
|
toast.error('Failed to archive project');
|
||||||
|
} finally {
|
||||||
|
setArchiving(false);
|
||||||
|
}
|
||||||
|
}}
|
||||||
|
>
|
||||||
|
{archiving ? 'Archiving...' : 'Archive'}
|
||||||
|
</AlertDialogAction>
|
||||||
|
</AlertDialogFooter>
|
||||||
|
</AlertDialogContent>
|
||||||
|
</AlertDialog>
|
||||||
</div>
|
</div>
|
||||||
);
|
);
|
||||||
}
|
}
|
||||||
|
|||||||
@@ -0,0 +1,254 @@
|
|||||||
|
'use client';
|
||||||
|
|
||||||
|
import { useState, useEffect } from 'react';
|
||||||
|
import { Button } from '@/components/ui/button';
|
||||||
|
import {
|
||||||
|
Dialog,
|
||||||
|
DialogContent,
|
||||||
|
DialogDescription,
|
||||||
|
DialogFooter,
|
||||||
|
DialogHeader,
|
||||||
|
DialogTitle,
|
||||||
|
} from '@/components/ui/dialog';
|
||||||
|
import {
|
||||||
|
AlertDialog,
|
||||||
|
AlertDialogAction,
|
||||||
|
AlertDialogCancel,
|
||||||
|
AlertDialogContent,
|
||||||
|
AlertDialogDescription,
|
||||||
|
AlertDialogFooter,
|
||||||
|
AlertDialogHeader,
|
||||||
|
AlertDialogTitle,
|
||||||
|
} from '@/components/ui/alert-dialog';
|
||||||
|
import { Input } from '@/components/ui/input';
|
||||||
|
import { Label } from '@/components/ui/label';
|
||||||
|
import { Textarea } from '@/components/ui/textarea';
|
||||||
|
import {
|
||||||
|
Select,
|
||||||
|
SelectContent,
|
||||||
|
SelectItem,
|
||||||
|
SelectTrigger,
|
||||||
|
SelectValue,
|
||||||
|
} from '@/components/ui/select';
|
||||||
|
import { toast } from 'sonner';
|
||||||
|
|
||||||
|
interface Project {
|
||||||
|
id: string;
|
||||||
|
name: string;
|
||||||
|
description: string | null;
|
||||||
|
status: 'active' | 'paused' | 'completed' | 'archived';
|
||||||
|
domainId: string;
|
||||||
|
color: string | null;
|
||||||
|
icon: string | null;
|
||||||
|
targetDate: string | null;
|
||||||
|
}
|
||||||
|
|
||||||
|
interface ProjectEditDialogProps {
|
||||||
|
open: boolean;
|
||||||
|
onOpenChange: (open: boolean) => void;
|
||||||
|
project: Project;
|
||||||
|
domainId: string;
|
||||||
|
onUpdated: () => void;
|
||||||
|
}
|
||||||
|
|
||||||
|
export function ProjectEditDialog({
|
||||||
|
open,
|
||||||
|
onOpenChange,
|
||||||
|
project,
|
||||||
|
domainId,
|
||||||
|
onUpdated,
|
||||||
|
}: ProjectEditDialogProps) {
|
||||||
|
const [name, setName] = useState('');
|
||||||
|
const [description, setDescription] = useState('');
|
||||||
|
const [status, setStatus] = useState<'active' | 'paused' | 'completed' | 'archived'>('active');
|
||||||
|
const [color, setColor] = useState('');
|
||||||
|
const [targetDate, setTargetDate] = useState('');
|
||||||
|
const [submitting, setSubmitting] = useState(false);
|
||||||
|
const [error, setError] = useState('');
|
||||||
|
const [archiveOpen, setArchiveOpen] = useState(false);
|
||||||
|
const [archiving, setArchiving] = useState(false);
|
||||||
|
|
||||||
|
useEffect(() => {
|
||||||
|
if (open && project) {
|
||||||
|
setName(project.name);
|
||||||
|
setDescription(project.description || '');
|
||||||
|
setStatus(project.status);
|
||||||
|
setColor(project.color || '');
|
||||||
|
setTargetDate(project.targetDate ? project.targetDate.split('T')[0] : '');
|
||||||
|
setError('');
|
||||||
|
}
|
||||||
|
}, [open, project]);
|
||||||
|
|
||||||
|
async function handleSubmit(event: React.FormEvent<HTMLFormElement>) {
|
||||||
|
event.preventDefault();
|
||||||
|
if (!domainId) {
|
||||||
|
setError('No domain selected');
|
||||||
|
return;
|
||||||
|
}
|
||||||
|
setSubmitting(true);
|
||||||
|
setError('');
|
||||||
|
|
||||||
|
const body: Record<string, unknown> = { name, status };
|
||||||
|
if (description) body.description = description;
|
||||||
|
if (color) body.color = color;
|
||||||
|
if (targetDate) body.targetDate = new Date(targetDate).toISOString();
|
||||||
|
|
||||||
|
try {
|
||||||
|
const response = await fetch(`/api/domains/${domainId}/projects/${project.id}`, {
|
||||||
|
method: 'PATCH',
|
||||||
|
headers: { 'Content-Type': 'application/json' },
|
||||||
|
body: JSON.stringify(body),
|
||||||
|
});
|
||||||
|
|
||||||
|
if (!response.ok) {
|
||||||
|
const err = await response.json();
|
||||||
|
throw new Error(err.error?.message || 'Unable to update project');
|
||||||
|
}
|
||||||
|
|
||||||
|
toast.success('Project updated');
|
||||||
|
onOpenChange(false);
|
||||||
|
onUpdated();
|
||||||
|
} catch (err) {
|
||||||
|
setError(err instanceof Error ? err.message : 'Unable to update project');
|
||||||
|
} finally {
|
||||||
|
setSubmitting(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
async function handleArchive() {
|
||||||
|
setArchiving(true);
|
||||||
|
try {
|
||||||
|
const response = await fetch(`/api/domains/${domainId}/projects/${project.id}`, {
|
||||||
|
method: 'PATCH',
|
||||||
|
headers: { 'Content-Type': 'application/json' },
|
||||||
|
body: JSON.stringify({ status: 'archived' }),
|
||||||
|
});
|
||||||
|
|
||||||
|
if (!response.ok) {
|
||||||
|
const err = await response.json();
|
||||||
|
throw new Error(err.error?.message || 'Unable to archive project');
|
||||||
|
}
|
||||||
|
|
||||||
|
toast.success('Project archived');
|
||||||
|
setArchiveOpen(false);
|
||||||
|
onOpenChange(false);
|
||||||
|
onUpdated();
|
||||||
|
} catch (err) {
|
||||||
|
toast.error(err instanceof Error ? err.message : 'Unable to archive project');
|
||||||
|
} finally {
|
||||||
|
setArchiving(false);
|
||||||
|
}
|
||||||
|
}
|
||||||
|
|
||||||
|
return (
|
||||||
|
<>
|
||||||
|
<Dialog open={open} onOpenChange={onOpenChange}>
|
||||||
|
<DialogContent className="sm:max-w-[500px]">
|
||||||
|
<DialogHeader>
|
||||||
|
<DialogTitle>Edit Project</DialogTitle>
|
||||||
|
<DialogDescription>Update your project details.</DialogDescription>
|
||||||
|
</DialogHeader>
|
||||||
|
<form className="space-y-4" onSubmit={handleSubmit}>
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="edit-project-name">Name *</Label>
|
||||||
|
<Input
|
||||||
|
id="edit-project-name"
|
||||||
|
value={name}
|
||||||
|
onChange={(e) => setName(e.target.value)}
|
||||||
|
placeholder="Project name"
|
||||||
|
autoFocus
|
||||||
|
required
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="edit-project-description">Description</Label>
|
||||||
|
<Textarea
|
||||||
|
id="edit-project-description"
|
||||||
|
value={description}
|
||||||
|
onChange={(e) => setDescription(e.target.value)}
|
||||||
|
placeholder="Optional description..."
|
||||||
|
rows={2}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="grid grid-cols-2 gap-4">
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="edit-project-status">Status</Label>
|
||||||
|
<Select value={status} onValueChange={(v) => setStatus(v as any)}>
|
||||||
|
<SelectTrigger id="edit-project-status">
|
||||||
|
<SelectValue />
|
||||||
|
</SelectTrigger>
|
||||||
|
<SelectContent>
|
||||||
|
<SelectItem value="active">Active</SelectItem>
|
||||||
|
<SelectItem value="paused">Paused</SelectItem>
|
||||||
|
<SelectItem value="completed">Completed</SelectItem>
|
||||||
|
<SelectItem value="archived">Archived</SelectItem>
|
||||||
|
</SelectContent>
|
||||||
|
</Select>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="edit-project-color">Color</Label>
|
||||||
|
<Input
|
||||||
|
id="edit-project-color"
|
||||||
|
type="color"
|
||||||
|
value={color}
|
||||||
|
onChange={(e) => setColor(e.target.value)}
|
||||||
|
className="h-10"
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
<div className="space-y-2">
|
||||||
|
<Label htmlFor="edit-project-target-date">Target date</Label>
|
||||||
|
<Input
|
||||||
|
id="edit-project-target-date"
|
||||||
|
type="date"
|
||||||
|
value={targetDate}
|
||||||
|
onChange={(e) => setTargetDate(e.target.value)}
|
||||||
|
/>
|
||||||
|
</div>
|
||||||
|
|
||||||
|
{error && <p className="text-sm text-destructive" role="alert">{error}</p>}
|
||||||
|
|
||||||
|
<DialogFooter className="flex items-center justify-between sm:justify-between">
|
||||||
|
<Button
|
||||||
|
type="button"
|
||||||
|
variant="destructive"
|
||||||
|
onClick={() => setArchiveOpen(true)}
|
||||||
|
>
|
||||||
|
Archive
|
||||||
|
</Button>
|
||||||
|
<div className="flex gap-2">
|
||||||
|
<Button type="button" variant="outline" onClick={() => onOpenChange(false)}>
|
||||||
|
Cancel
|
||||||
|
</Button>
|
||||||
|
<Button type="submit" disabled={submitting || !name || !domainId}>
|
||||||
|
{submitting ? 'Saving...' : 'Save'}
|
||||||
|
</Button>
|
||||||
|
</div>
|
||||||
|
</DialogFooter>
|
||||||
|
</form>
|
||||||
|
</DialogContent>
|
||||||
|
</Dialog>
|
||||||
|
|
||||||
|
<AlertDialog open={archiveOpen} onOpenChange={setArchiveOpen}>
|
||||||
|
<AlertDialogContent>
|
||||||
|
<AlertDialogHeader>
|
||||||
|
<AlertDialogTitle>Archive Project</AlertDialogTitle>
|
||||||
|
<AlertDialogDescription>
|
||||||
|
Are you sure you want to archive "{project.name}"? It will be hidden from the active list.
|
||||||
|
</AlertDialogDescription>
|
||||||
|
</AlertDialogHeader>
|
||||||
|
<AlertDialogFooter>
|
||||||
|
<AlertDialogCancel>Cancel</AlertDialogCancel>
|
||||||
|
<AlertDialogAction onClick={handleArchive} disabled={archiving}>
|
||||||
|
{archiving ? 'Archiving...' : 'Archive'}
|
||||||
|
</AlertDialogAction>
|
||||||
|
</AlertDialogFooter>
|
||||||
|
</AlertDialogContent>
|
||||||
|
</AlertDialog>
|
||||||
|
</>
|
||||||
|
);
|
||||||
|
}
|
||||||
Reference in New Issue
Block a user