2026-05-31 15:06:10 -05:00
|
|
|
import { useState } from 'react';
|
|
|
|
|
import { toast } from 'sonner';
|
refactor(ts): convert the API client to TypeScript (TS9)
client/api.js -> api.ts. Types the fetch infrastructure: generic _fetch<T>
and get/post/put/patch/del<T> helpers, an ApiError interface (status/code/
details/data), a QueryParams type for the query-string builder, and File-typed
upload helpers. Endpoint response shapes are typed incrementally — most methods
default to Promise<unknown>; the ones consumed by typed .ts files get real
shapes (quickPay -> PaymentRecord, togglePaid -> TogglePaidResult, settings ->
settings map).
Typing togglePaid's result surfaced a latent narrowing bug in usePaymentActions:
the un-pay Undo closure read result.paymentId (number|undefined) inside a
deferred async callback where TS re-widens the if-narrowed value — fixed by
capturing the id in a const before the closure.
The 16 files importing '@/api.js' with an explicit extension were normalized to
'@/api' so Vite/TS resolve the .ts.
Verified: typecheck 0, lint 0 errors (47 warns), build green, 48 client tests,
and 17/17 e2e probe (every page renders, all API paths respond) — the central
fetch-module rename is runtime-safe.
Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
2026-07-03 22:09:04 -05:00
|
|
|
import { api } from '@/api';
|
2026-05-31 15:06:10 -05:00
|
|
|
import { cn, fmt } from '@/lib/utils';
|
|
|
|
|
import { Button } from '@/components/ui/button';
|
|
|
|
|
import { MONTHS, rowThreshold, paymentSummary } from '@/lib/trackerUtils';
|
|
|
|
|
|
|
|
|
|
export function LowerThisMonthButton({ row, year, month, refresh, compact = false }) {
|
|
|
|
|
const threshold = rowThreshold(row);
|
|
|
|
|
const summary = paymentSummary(row, threshold);
|
|
|
|
|
const [saving, setSaving] = useState(false);
|
|
|
|
|
|
|
|
|
|
if (row.is_skipped || !summary.partial) return null;
|
|
|
|
|
|
|
|
|
|
async function handleClick() {
|
|
|
|
|
setSaving(true);
|
|
|
|
|
try {
|
|
|
|
|
await api.saveBillMonthlyState(row.id, {
|
|
|
|
|
year,
|
|
|
|
|
month,
|
|
|
|
|
actual_amount: summary.paid,
|
|
|
|
|
notes: row.monthly_notes || null,
|
|
|
|
|
is_skipped: row.is_skipped,
|
|
|
|
|
});
|
|
|
|
|
toast.success(`${MONTHS[month - 1]} amount set to ${fmt(summary.paid)}`);
|
|
|
|
|
refresh?.();
|
|
|
|
|
} catch (err) {
|
|
|
|
|
toast.error(err.message || 'Failed to update monthly amount');
|
|
|
|
|
} finally {
|
|
|
|
|
setSaving(false);
|
|
|
|
|
}
|
|
|
|
|
}
|
|
|
|
|
|
|
|
|
|
return (
|
|
|
|
|
<Button
|
|
|
|
|
type="button"
|
|
|
|
|
size="sm"
|
|
|
|
|
variant="ghost"
|
|
|
|
|
disabled={saving}
|
|
|
|
|
onClick={handleClick}
|
|
|
|
|
className={cn(
|
|
|
|
|
'h-7 px-2.5 text-xs font-semibold text-amber-600 hover:bg-amber-500/10 hover:text-amber-700',
|
|
|
|
|
'dark:text-amber-400 dark:hover:text-amber-300',
|
|
|
|
|
compact && 'h-8',
|
|
|
|
|
)}
|
|
|
|
|
title={`Set ${MONTHS[month - 1]} amount to ${fmt(summary.paid)} because this bill was lower this month`}
|
|
|
|
|
>
|
|
|
|
|
{saving ? 'Saving...' : 'Bill was lower'}
|
|
|
|
|
</Button>
|
|
|
|
|
);
|
|
|
|
|
}
|