Improve shop hub: CurrencyIcon on packages/vouchers, DataTable for transactions.

Co-authored-by: Cursor <[email protected]>

src/actions/admin-vouchers.ts

@@ -1,30 +1,42 @@
 "use server";
 
 import { revalidatePath } from "next/cache";
+import { positiveBigInt } from "@/lib/api";
 import { requirePermission } from "@/lib/admin/guard";
 import { PERMS } from "@/lib/permissions";
-import { formPositiveBigInt } from "@/lib/form-data";
 import { prisma } from "@/lib/prisma";
+import {
+	actionError,
+	actionOk,
+	type ActionResult,
+} from "@/lib/safe-action-shared";
 import { logServerError } from "@/lib/server-log";
 
-export async function createVoucher(formData: FormData): Promise<void> {
+export async function createVoucher(input: {
+	code: string;
+	amount: number;
+	maxUses: number;
+	expiresAt?: string;
+}): Promise<ActionResult<{ id: string }>> {
 	await requirePermission(PERMS.SHOP_EDIT);
 
-	const code = String(formData.get("code") ?? "")
+	const code = String(input.code ?? "")
 		.normalize("NFC")
 		.trim()
 		.slice(0, 255);
-	const amount = Number(formData.get("amount"));
-	const maxUsesRaw = Number(formData.get("maxUses"));
+	const amount = Number(input.amount);
+	const maxUsesRaw = Number(input.maxUses);
 	const maxUses =
 		Number.isFinite(maxUsesRaw) && maxUsesRaw > 0 ? Math.floor(maxUsesRaw) : 1;
 
-	if (!code || !(amount > 0)) return;
+	if (!code || !(amount > 0)) {
+		return actionError("Code and a positive amount are required");
+	}
 
-	const expiresRaw = String(formData.get("expiresAt") ?? "")
+	let expiresAt: Date | null = null;
+	const expiresRaw = String(input.expiresAt ?? "")
 		.normalize("NFC")
 		.trim();
-	let expiresAt: Date | null = null;
 	if (expiresRaw) {
 		const parsed = new Date(expiresRaw);
 		if (!Number.isNaN(parsed.getTime())) expiresAt = parsed;
@@ -33,7 +45,7 @@ export async function createVoucher(formData: FormData): Promise<void> {
 	const now = new Date();
 
 	try {
-		await prisma.websiteShopVouchers.create({
+		const created = await prisma.websiteShopVouchers.create({
 			data: {
 				code,
 				amount: Math.floor(amount),
@@ -44,29 +56,30 @@ export async function createVoucher(formData: FormData): Promise<void> {
 				updatedAt: now,
 			},
 		});
+		revalidatePath("/admin/vouchers");
+		return actionOk({ id: String(created.id) });
 	} catch (error) {
 		logServerError("admin.voucher_create_failed", error);
-		// Unique constraint on `code` (or DB unavailable) — swallow and re-render.
-		return;
+		return actionError("Could not create voucher (code may already exist)");
 	}
-
-	revalidatePath("/admin/vouchers");
 }
 
-export async function deleteVoucher(formData: FormData): Promise<void> {
+export async function deleteVoucher(input: {
+	id: string;
+}): Promise<ActionResult> {
 	await requirePermission(PERMS.SHOP_EDIT);
 
-	const id = formPositiveBigInt(formData, "id");
-	if (!id) return;
+	const id = positiveBigInt(String(input.id ?? "").trim());
+	if (!id) return actionError("Missing voucher id");
 
 	try {
 		await prisma.websiteShopVouchers.delete({ where: { id } });
+		revalidatePath("/admin/vouchers");
+		return actionOk();
 	} catch (error) {
 		logServerError("admin.voucher_delete_failed", error, {
 			voucherId: String(id),
 		});
-		return;
+		return actionError("Could not delete voucher");
 	}
-
-	revalidatePath("/admin/vouchers");
 }