Files
menassit/client/src/pages/FinancesPage.tsx
T
ghaddaditw 56f2f98a5e Improve financial record creation by ensuring data is handled correctly
Refactor createRecordMutation in FinancesPage.tsx to handle fetch responses properly.

Replit-Commit-Author: Agent
Replit-Commit-Session-Id: ff0be73b-afdd-4747-978b-bb8301fb0a82
Replit-Commit-Screenshot-Url: https://storage.googleapis.com/screenshot-production-us-central1/9777c70b-fc38-4831-8d6b-78dfffe041b0/1999d076-38ec-4633-8667-1740c231dafc.jpg
2025-06-08 00:06:16 +00:00

287 lines
12 KiB
TypeScript

import { useState } from "react";
import { useQuery, useMutation, useQueryClient } from "@tanstack/react-query";
import { Card, CardContent, CardDescription, CardHeader, CardTitle } from "@/components/ui/card";
import { Button } from "@/components/ui/button";
import { Input } from "@/components/ui/input";
import { Label } from "@/components/ui/label";
import { Select, SelectContent, SelectItem, SelectTrigger, SelectValue } from "@/components/ui/select";
import { Badge } from "@/components/ui/badge";
import { useToast } from "@/hooks/use-toast";
import Sidebar from "@/components/layout/Sidebar";
import { Plus, DollarSign, TrendingUp, TrendingDown } from "lucide-react";
import { Dialog, DialogContent, DialogDescription, DialogHeader, DialogTitle, DialogTrigger } from "@/components/ui/dialog";
import { apiRequest } from "@/lib/queryClient";
export default function FinancesPage() {
const { toast } = useToast();
const queryClient = useQueryClient();
const [open, setOpen] = useState(false);
const [newRecord, setNewRecord] = useState({
type: "expense",
amount: "",
description: "",
category: "",
});
const { data: recordsResponse, isLoading: recordsLoading } = useQuery({
queryKey: ["/api/financial/records"],
});
const { data: summaryResponse, isLoading: summaryLoading } = useQuery({
queryKey: ["/api/financial/summary"],
});
const records = (recordsResponse as any)?.records || [];
const summary = (summaryResponse as any)?.summary || { income: 0, expenses: 0, net: 0 };
const createRecordMutation = useMutation({
mutationFn: async (record: typeof newRecord) => {
const response = await fetch("/api/financial/records", {
method: "POST",
headers: {
"Content-Type": "application/json",
},
body: JSON.stringify({
...record,
amount: parseFloat(record.amount),
}),
credentials: "include",
});
if (!response.ok) throw new Error("Failed to create record");
return response.json();
},
onSuccess: () => {
queryClient.invalidateQueries({ queryKey: ["/api/financial/records"] });
queryClient.invalidateQueries({ queryKey: ["/api/financial/summary"] });
setOpen(false);
setNewRecord({ type: "expense", amount: "", description: "", category: "" });
toast({
title: "Record created",
description: "Your financial record has been successfully created.",
});
},
onError: () => {
toast({
title: "Error",
description: "Failed to create record. Please try again.",
variant: "destructive",
});
},
});
const handleCreateRecord = (e: React.FormEvent) => {
e.preventDefault();
if (!newRecord.amount || isNaN(parseFloat(newRecord.amount))) {
toast({
title: "Invalid amount",
description: "Please enter a valid amount.",
variant: "destructive",
});
return;
}
createRecordMutation.mutate(newRecord);
};
const categories = {
income: ["Salary", "Freelance", "Investment", "Business", "Other"],
expense: ["Food", "Transportation", "Housing", "Entertainment", "Healthcare", "Shopping", "Other"],
};
if (recordsLoading || summaryLoading) {
return (
<div className="flex h-screen bg-gray-50 dark:bg-gray-900">
<Sidebar className="w-64 border-r" />
<div className="flex-1 overflow-auto p-6">
<div className="animate-pulse space-y-4">
<div className="h-8 bg-gray-200 dark:bg-gray-700 rounded w-1/4"></div>
<div className="h-4 bg-gray-200 dark:bg-gray-700 rounded w-1/2"></div>
<div className="grid grid-cols-3 gap-4">
{[1, 2, 3].map((i) => (
<div key={i} className="h-24 bg-gray-200 dark:bg-gray-700 rounded"></div>
))}
</div>
</div>
</div>
</div>
);
}
return (
<div className="flex h-screen bg-gray-50 dark:bg-gray-900">
<Sidebar className="w-64 border-r" />
<div className="flex-1 overflow-auto">
<div className="p-6">
<div className="flex justify-between items-center mb-6">
<div>
<h1 className="text-2xl font-bold text-gray-900 dark:text-white">Finances</h1>
<p className="text-gray-600 dark:text-gray-300">
Track your income, expenses, and financial goals
</p>
</div>
<Dialog open={open} onOpenChange={setOpen}>
<DialogTrigger asChild>
<Button>
<Plus className="w-4 h-4 mr-2" />
Add Record
</Button>
</DialogTrigger>
<DialogContent>
<DialogHeader>
<DialogTitle>Add Financial Record</DialogTitle>
<DialogDescription>
Add a new income or expense record.
</DialogDescription>
</DialogHeader>
<form onSubmit={handleCreateRecord} className="space-y-4">
<div className="space-y-2">
<Label htmlFor="type">Type</Label>
<Select value={newRecord.type} onValueChange={(value) => setNewRecord({ ...newRecord, type: value, category: "" })}>
<SelectTrigger>
<SelectValue />
</SelectTrigger>
<SelectContent>
<SelectItem value="income">Income</SelectItem>
<SelectItem value="expense">Expense</SelectItem>
</SelectContent>
</Select>
</div>
<div className="space-y-2">
<Label htmlFor="amount">Amount</Label>
<Input
id="amount"
type="number"
step="0.01"
placeholder="0.00"
value={newRecord.amount}
onChange={(e) => setNewRecord({ ...newRecord, amount: e.target.value })}
required
/>
</div>
<div className="space-y-2">
<Label htmlFor="category">Category</Label>
<Select value={newRecord.category} onValueChange={(value) => setNewRecord({ ...newRecord, category: value })}>
<SelectTrigger>
<SelectValue placeholder="Select category" />
</SelectTrigger>
<SelectContent>
{categories[newRecord.type as keyof typeof categories].map((category) => (
<SelectItem key={category} value={category}>
{category}
</SelectItem>
))}
</SelectContent>
</Select>
</div>
<div className="space-y-2">
<Label htmlFor="description">Description</Label>
<Input
id="description"
placeholder="Brief description"
value={newRecord.description}
onChange={(e) => setNewRecord({ ...newRecord, description: e.target.value })}
/>
</div>
<Button type="submit" disabled={createRecordMutation.isPending}>
{createRecordMutation.isPending ? "Adding..." : "Add Record"}
</Button>
</form>
</DialogContent>
</Dialog>
</div>
{/* Summary Cards */}
<div className="grid grid-cols-1 md:grid-cols-3 gap-6 mb-6">
<Card>
<CardHeader className="flex flex-row items-center justify-between space-y-0 pb-2">
<CardTitle className="text-sm font-medium">Total Income</CardTitle>
<TrendingUp className="h-4 w-4 text-green-600" />
</CardHeader>
<CardContent>
<div className="text-2xl font-bold text-green-600">
${summary.income.toFixed(2)}
</div>
</CardContent>
</Card>
<Card>
<CardHeader className="flex flex-row items-center justify-between space-y-0 pb-2">
<CardTitle className="text-sm font-medium">Total Expenses</CardTitle>
<TrendingDown className="h-4 w-4 text-red-600" />
</CardHeader>
<CardContent>
<div className="text-2xl font-bold text-red-600">
${summary.expenses.toFixed(2)}
</div>
</CardContent>
</Card>
<Card>
<CardHeader className="flex flex-row items-center justify-between space-y-0 pb-2">
<CardTitle className="text-sm font-medium">Net Balance</CardTitle>
<DollarSign className="h-4 w-4 text-blue-600" />
</CardHeader>
<CardContent>
<div className={`text-2xl font-bold ${summary.net >= 0 ? 'text-green-600' : 'text-red-600'}`}>
${summary.net.toFixed(2)}
</div>
</CardContent>
</Card>
</div>
{/* Records List */}
<Card>
<CardHeader>
<CardTitle>Recent Transactions</CardTitle>
<CardDescription>
Your latest financial transactions
</CardDescription>
</CardHeader>
<CardContent>
{records.length === 0 ? (
<div className="text-center py-6">
<DollarSign className="w-12 h-12 mx-auto text-gray-400 mb-4" />
<h3 className="text-lg font-semibold text-gray-900 dark:text-white mb-2">
No records yet
</h3>
<p className="text-gray-600 dark:text-gray-300 mb-4">
Start tracking your finances by adding your first record.
</p>
<Button onClick={() => setOpen(true)}>
<Plus className="w-4 h-4 mr-2" />
Add Record
</Button>
</div>
) : (
<div className="space-y-4">
{records.map((record: any) => (
<div key={record.id} className="flex items-center justify-between p-4 border rounded-lg">
<div className="flex items-center gap-4">
<div className={`p-2 rounded-full ${record.type === 'income' ? 'bg-green-100 text-green-600' : 'bg-red-100 text-red-600'}`}>
{record.type === 'income' ? <TrendingUp className="w-4 h-4" /> : <TrendingDown className="w-4 h-4" />}
</div>
<div>
<div className="font-medium">{record.description || record.category}</div>
<div className="text-sm text-gray-600 dark:text-gray-300">
{record.category} {new Date(record.createdAt).toLocaleDateString()}
</div>
</div>
</div>
<div className="text-right">
<div className={`font-semibold ${record.type === 'income' ? 'text-green-600' : 'text-red-600'}`}>
{record.type === 'income' ? '+' : '-'}${record.amount.toFixed(2)}
</div>
<Badge variant={record.type === 'income' ? 'default' : 'secondary'}>
{record.type}
</Badge>
</div>
</div>
))}
</div>
)}
</CardContent>
</Card>
</div>
</div>
</div>
);
}