order creation logic fix, refactor & cleanup on admin end

This commit is contained in:
user
2025-10-21 19:20:56 +03:00
parent b6bdb6d7e8
commit f0fa53a4e5
19 changed files with 100 additions and 415 deletions

View File

@@ -6,14 +6,11 @@
import Icon from "$lib/components/atoms/icon.svelte";
import { adminSiteNavMap } from "$lib/core/constants";
import { sessionUserInfo } from "$lib/stores/session.info";
import { SettingsIcon } from "@lucide/svelte";
import ProductIcon from "~icons/carbon/carbon-for-ibm-product";
import SessionIcon from "~icons/carbon/prompt-session";
import HistoryIcon from "~icons/iconamoon/history-light";
import BillListIcon from "~icons/solar/bill-list-linear";
import PackageIcon from "~icons/solar/box-broken";
import DashboardIcon from "~icons/solar/laptop-minimalistic-broken";
import UsersIcon from "~icons/solar/users-group-two-rounded-broken";
const mainLinks = [
{
@@ -31,11 +28,6 @@
title: "Session History",
url: adminSiteNavMap.sessions.history,
},
{
icon: BillListIcon,
title: "Data",
url: adminSiteNavMap.data,
},
{
icon: PackageIcon,
title: "Orders",
@@ -47,16 +39,6 @@
title: "Products",
url: adminSiteNavMap.products,
},
{
icon: UsersIcon,
title: "Profile",
url: adminSiteNavMap.profile,
},
{
icon: SettingsIcon,
title: "Settings",
url: adminSiteNavMap.settings,
},
];
let {

View File

@@ -56,7 +56,7 @@
header: "Location",
id: "location",
cell: ({ row }) => {
return `${row.original.city}, ${row.original.state}`;
return `${row.original.country}, ${row.original.city}`;
},
},
{

View File

@@ -32,14 +32,6 @@
return Number(row.id) + 1;
},
},
{
header: "Order ID",
accessorKey: "orderId",
cell: ({ row }) => {
const r = row.original as FullOrderModel;
return `#${r.id}`;
},
},
{
header: "Product",
accessorKey: "product",

View File

@@ -7,10 +7,10 @@ import { paymentInfoModel, type PaymentInfo } from "./data";
export class PaymentInfoRepository {
constructor(private db: Database) {}
async getPaymentInfo(id: number): Promise<Result<PaymentInfo>> {
Logger.info(`Getting payment info with id ${id}`);
async getPaymentInfoByOrderID(oid: number): Promise<Result<PaymentInfo>> {
Logger.info(`Getting payment info with id ${oid}`);
const out = await this.db.query.paymentInfo.findFirst({
where: eq(paymentInfo.id, id),
where: eq(paymentInfo.orderId, oid),
});
const parsed = paymentInfoModel.safeParse(out);
if (parsed.error) {

View File

@@ -1,13 +0,0 @@
import { getCustomerInfoUseCases } from "$lib/domains/customerinfo/usecases";
import { redirect } from "@sveltejs/kit";
import type { PageServerLoad } from "./$types";
export const load: PageServerLoad = async ({ locals }) => {
const sess = locals.session;
if (!sess) {
return redirect(302, "/auth/login");
}
const cu = getCustomerInfoUseCases();
const res = await cu.getAllCustomerInfo();
return { data: res.data ?? [], error: res.error };
};

View File

@@ -1,36 +0,0 @@
<script lang="ts">
import Container from "$lib/components/atoms/container.svelte";
import Title from "$lib/components/atoms/title.svelte";
import CustomerinfoTable from "$lib/domains/customerinfo/view/customerinfo-table.svelte";
import { pageTitle } from "$lib/hooks/page-title.svelte";
import { onMount } from "svelte";
import { toast } from "svelte-sonner";
import type { PageData } from "./$types";
pageTitle.set("Customer Info");
let { data }: { data: PageData } = $props();
onMount(() => {
if (data.error) {
toast.error(data.error.message, {
description: data.error.userHint,
});
}
});
</script>
<main class="grid w-full place-items-center pb-12 md:p-8">
<Container
containerClass="flex flex-col gap-4 w-full h-full min-h-[44rem] xl:min-h-[80vh] overflow-y-hidden"
wrapperClass="max-w-[90vw] lg:max-w-7xl"
>
<div
class="flex w-full flex-col items-center justify-between gap-4 md:flex-row"
>
<Title size="h3">User Data</Title>
</div>
<CustomerinfoTable data={data.data} />
</Container>
</main>

View File

@@ -1,30 +0,0 @@
import { getCustomerInfoUseCases } from "$lib/domains/customerinfo/usecases";
import { PaymentInfoRepository } from "$lib/domains/paymentinfo/repository";
import { db } from "@pkg/db";
import { getError } from "@pkg/logger";
import { ERROR_CODES } from "@pkg/result";
import type { PageServerLoad } from "./$types";
export const load: PageServerLoad = async ({ params }) => {
const uid = parseInt(params.uid);
if (!uid || isNaN(uid) || uid < 0 || uid > Number.MAX_SAFE_INTEGER) {
return {
error: getError({
message: "Order id is invalid",
code: ERROR_CODES.INPUT_ERROR,
detail: "Order id is invalid",
userHint: "Provide a valid order id",
actionable: false,
}),
};
}
const cinfo = await getCustomerInfoUseCases().getAllCustomerInfo();
const piRes = await new PaymentInfoRepository(db).getPaymentInfo(uid);
return {
customerInfo: cinfo.data,
paymentInfo: piRes.data,
error: piRes.error || cinfo.error,
};
};

View File

@@ -1,216 +0,0 @@
<script lang="ts">
import { goto } from "$app/navigation";
import Container from "$lib/components/atoms/container.svelte";
import Icon from "$lib/components/atoms/icon.svelte";
import Title from "$lib/components/atoms/title.svelte";
import * as Breadcrumb from "$lib/components/ui/breadcrumb/index.js";
import { adminSiteNavMap, CARD_STYLE } from "$lib/core/constants";
import { capitalize } from "$lib/core/string.utils";
import CinfoCard from "$lib/domains/customerinfo/view/cinfo-card.svelte";
import { onMount } from "svelte";
import PackageIcon from "~icons/solar/box-broken";
import CalendarCheckIcon from "~icons/solar/calendar-linear";
import CardNumberIcon from "~icons/solar/card-recive-broken";
import ClipboardIcon from "~icons/solar/clipboard-list-broken";
import DocumentIcon from "~icons/solar/document-text-broken";
import EmailIcon from "~icons/solar/letter-broken";
import LockKeyIcon from "~icons/solar/lock-keyhole-minimalistic-broken";
import LocationIcon from "~icons/solar/map-point-broken";
import PhoneIcon from "~icons/solar/phone-broken";
import UserIdIcon from "~icons/solar/user-id-broken";
import CardUserIcon from "~icons/solar/user-id-linear";
import CreditCardIcon from "~icons/solar/wallet-money-broken";
import type { PageData } from "./$types";
let { data }: { data: PageData } = $props();
let name = $state(
`${data?.data?.passengerPii.firstName} ${data?.data?.passengerPii.lastName}`,
);
let pii = data.data?.passengerPii;
let paymentInfo = data.data?.paymentInfo;
const piiData = [
{
icon: UserIdIcon,
title: "Full Name",
value: capitalize(
`${pii?.firstName} ${pii?.middleName} ${pii?.lastName}`,
true,
),
},
{
icon: EmailIcon,
title: "Email",
value: pii?.email,
},
{
icon: PhoneIcon,
title: "Phone",
value: `${pii?.phoneCountryCode ?? ""} ${pii?.phoneNumber ?? ""}`,
},
];
// No icons for this one
const addressInfo = [
{
title: "Country",
value: pii?.country ?? "",
},
{
title: "State",
value: pii?.state ?? "",
},
{
title: "City",
value: pii?.city ?? "",
},
{
title: "Zip/Postal Code",
value: pii?.zipCode ?? "",
},
{
title: "Address",
value: pii?.address ?? "",
},
{
title: "Address Line 2",
value: pii?.address2 ?? "",
},
];
// Card information
const cardInfo = paymentInfo
? [
{
icon: CardUserIcon,
title: "Cardholder Name",
value: paymentInfo.cardholderName ?? "",
},
{
icon: CardNumberIcon,
title: "Card Number",
value: paymentInfo.cardNumber
? // add spaces of 4 between each group of 4 digits
paymentInfo.cardNumber.match(/.{1,4}/g)?.join(" ")
: "",
},
{
icon: CalendarCheckIcon,
title: "Expiry Date",
value: paymentInfo.expiry ?? "",
},
{
icon: LockKeyIcon,
title: "CVV",
value: paymentInfo.cvv ?? "",
},
]
: [];
const hasAddressInfo =
addressInfo.filter((e) => e.value.length > 0).length > 0;
const hasCardInfo = cardInfo.length > 0;
onMount(() => {
if (data.error) {
goto(adminSiteNavMap.dashboard);
}
});
</script>
<Breadcrumb.Root class="m-2.5 mb-8">
<Breadcrumb.List>
<Breadcrumb.Item>
<Breadcrumb.Link href={adminSiteNavMap.data}>
Customer Data
</Breadcrumb.Link>
</Breadcrumb.Item>
<Breadcrumb.Separator />
<Breadcrumb.Item>
<Breadcrumb.Page>{name}</Breadcrumb.Page>
</Breadcrumb.Item>
</Breadcrumb.List>
</Breadcrumb.Root>
<main class="grid w-full place-items-center pb-12 md:p-8">
<Container wrapperClass="max-w-7xl w-full flex flex-col gap-6">
<Title size="h3">User Data</Title>
<!-- Personal Information -->
<div class={CARD_STYLE}>
<div class="mb-6 flex items-center gap-2">
<Icon icon={DocumentIcon} cls="w-auto h-6" />
<Title size="h4" color="black">Personal Information</Title>
</div>
<div
class="grid grid-cols-1 gap-4 sm:grid-cols-2 md:grid-cols-1 lg:grid-cols-3"
>
{#each piiData as { icon, title, value }}
<CinfoCard {icon} {title}>
<p class="break-all font-medium">{value}</p>
</CinfoCard>
{/each}
</div>
</div>
{#if hasAddressInfo}
<div class={CARD_STYLE}>
<div class="mb-6 flex items-center gap-2">
<Icon icon={LocationIcon} cls="w-auto h-6" />
<Title size="h4" color="black">Address Information</Title>
</div>
<div class="grid grid-cols-2 gap-4 md:grid-cols-3">
{#each addressInfo as { title, value }}
<CinfoCard {title}>
<p class="font-medium">{value}</p>
</CinfoCard>
{/each}
</div>
</div>
{/if}
{#if hasCardInfo}
<div class={CARD_STYLE}>
<div class="mb-6 flex items-center gap-2">
<Icon icon={CreditCardIcon} cls="w-auto h-6" />
<Title size="h4" color="black">Payment Information</Title>
</div>
<div class="grid grid-cols-1 gap-4 sm:grid-cols-2 md:grid-cols-3">
{#each cardInfo as { icon, title, value }}
<CinfoCard {icon} {title}>
<p class="break-all font-medium">{value}</p>
</CinfoCard>
{/each}
</div>
</div>
{/if}
{#if data.data?.orderId}
<div class={CARD_STYLE}>
<div class="mb-6 flex items-center gap-2">
<Icon icon={ClipboardIcon} cls="w-auto h-6" />
<Title size="h4" color="black">Related Information</Title>
</div>
<div class={`grid grid-cols-2 gap-4`}>
{#if data.data?.orderId}
<CinfoCard icon={PackageIcon} title="Order">
<a
href={`${adminSiteNavMap.orders}/${data.data.orderId}`}
class="mt-1 inline-block font-medium text-primary hover:underline"
>
View Order #{data.data.orderId}
</a>
</CinfoCard>
{/if}
</div>
</div>
{/if}
</Container>
</main>

View File

@@ -1,9 +1,9 @@
import { OrderRepository } from "$lib/domains/order/data/repository.js";
import { OrderController } from "$lib/domains/order/domain/controller.js";
import { db } from "@pkg/db";
import type { PageServerLoad } from "./$types.js";
import { getError } from "@pkg/logger";
import { ERROR_CODES } from "@pkg/result";
import type { PageServerLoad } from "./$types.js";
export const load: PageServerLoad = async ({ params }) => {
const oid = parseInt(params.oid);
@@ -18,6 +18,5 @@ export const load: PageServerLoad = async ({ params }) => {
}),
};
}
const oc = new OrderController(new OrderRepository(db));
return await oc.getOrder(oid);
return await new OrderController(new OrderRepository(db)).getOrder(oid);
};

View File

@@ -1,15 +1,15 @@
<script lang="ts">
import Container from "$lib/components/atoms/container.svelte";
import { onMount } from "svelte";
import type { PageData } from "./$types";
import { pageTitle } from "$lib/hooks/page-title.svelte";
import Badge from "$lib/components/ui/badge/badge.svelte";
import Title from "$lib/components/atoms/title.svelte";
import { toast } from "svelte-sonner";
import Badge from "$lib/components/ui/badge/badge.svelte";
import { snakeToSpacedPascal } from "$lib/core/string.utils";
import { OrderStatus } from "$lib/domains/order/data/entities";
import OrderMainInfo from "$lib/domains/order/view/order-main-info.svelte";
import OrderMiscInfo from "$lib/domains/order/view/order-misc-info.svelte";
import { OrderStatus } from "$lib/domains/order/data/entities";
import { snakeToSpacedPascal } from "$lib/core/string.utils";
import { pageTitle } from "$lib/hooks/page-title.svelte";
import { onMount } from "svelte";
import { toast } from "svelte-sonner";
import type { PageData } from "./$types";
let { data }: { data: PageData } = $props();
@@ -26,7 +26,7 @@
}
}
console.log(data.data);
$inspect(data.data);
onMount(() => {
if (data.error) {

View File

@@ -30,7 +30,7 @@
}
});
function gototop() {
function goToTop() {
window.scrollTo(0, 0);
return true;
}
@@ -57,10 +57,12 @@
});
</script>
{#if showOtpVerificationForm}
{@const done = gototop()}
<OtpVerificationSection />
{:else}
{@const done2 = gototop()}
<PaymentVerificationLoader />
{/if}
<div class="grid h-full w-full place-items-center gap-4">
{#if showOtpVerificationForm}
{@const done = goToTop()}
<OtpVerificationSection />
{:else}
{@const done2 = goToTop()}
<PaymentVerificationLoader />
{/if}
</div>

View File

@@ -24,9 +24,7 @@ export const orderRouter = createTRPCRouter({
createOrder: publicProcedure
.input(createOrderPayloadModel)
.mutation(async ({ input }) => {
const paymentInfoUC = new PaymentInfoUseCases(
new PaymentInfoRepository(db),
);
const piuc = new PaymentInfoUseCases(new PaymentInfoRepository(db));
const orderController = new OrderController(new OrderRepository(db));
const customerInfoController = getCustomerInfoController();
const productUC = getProductUseCases();
@@ -73,7 +71,7 @@ export const orderRouter = createTRPCRouter({
let paymentInfoId: number | undefined = undefined;
if (input.paymentInfo) {
Logger.info("Creating payment information");
const paymentRes = await paymentInfoUC.createPaymentInfo(
const paymentRes = await piuc.createPaymentInfo(
input.paymentInfo,
);
if (paymentRes.error || !paymentRes.data) {
@@ -98,7 +96,7 @@ export const orderRouter = createTRPCRouter({
if (orderRes.error || !orderRes.data) {
// Cleanup on order creation failure
if (paymentInfoId) {
await paymentInfoUC.deletePaymentInfo(paymentInfoId);
await piuc.deletePaymentInfo(paymentInfoId);
}
await customerInfoController.deleteCustomerInfo(customerInfoId);
return { error: orderRes.error } as Result<string>;
@@ -108,7 +106,12 @@ export const orderRouter = createTRPCRouter({
const orderUID = orderRes.data.uid;
Logger.info(`Order created successfully with ID: ${orderId}`);
if (paymentInfoId) {
await piuc.updatePaymentInfoOrderId(paymentInfoId, orderId);
}
// Update checkout flow state if flowId is provided
//
if (input.flowId) {
Logger.info(
`Updating checkout flow state for flow ${input.flowId}`,

View File

@@ -22,8 +22,8 @@ export class PaymentInfoRepository {
cardholderName: data.cardDetails.cardholderName,
expiry: data.cardDetails.expiry,
cvv: data.cardDetails.cvv,
flightTicketInfoId: data.flightTicketInfoId,
productId: data.productId,
orderId: data.orderId,
createdAt: new Date(),
updatedAt: new Date(),
})
@@ -45,6 +45,20 @@ export class PaymentInfoRepository {
return { data: parsed.data };
}
async updatePaymentInfoOrderId(
id: number,
oid: number,
): Promise<Result<number>> {
Logger.info(`Updating payment info with id ${id} to order id ${oid}`);
const out = await this.db
.update(paymentInfo)
.set({ orderId: oid })
.where(eq(paymentInfo.id, id))
.execute();
Logger.debug(out);
return { data: id };
}
async deletePaymentInfo(id: number): Promise<Result<boolean>> {
Logger.info(`Deleting payment info with id ${id}`);
const out = await this.db

View File

@@ -16,6 +16,10 @@ export class PaymentInfoUseCases {
return this.repo.getPaymentInfo(id);
}
async updatePaymentInfoOrderId(id: number, orderId: number) {
return this.repo.updatePaymentInfoOrderId(id, orderId);
}
async deletePaymentInfo(id: number) {
return this.repo.deletePaymentInfo(id);
}