Make products table more consistent with other tables

apps/portal/src/lib/printful.ts

@@ -1,7 +0,0 @@
-import { PrintfulClient } from "@blade-and-brawn/commerce";
-import { env } from "$env/dynamic/private";
-
-export const printful = new PrintfulClient({
-    token: env.PRINTFUL_AUTH,
-    storeId: env.PRINTFUL_STORE_ID
-});

apps/portal/src/lib/webflow.ts

@@ -1,12 +0,0 @@
-import { WebflowClient } from "@blade-and-brawn/commerce";
-import { env } from "$env/dynamic/private";
-
-export const webflow = new WebflowClient({
-    siteId: env.WEBFLOW_SITE_ID,
-    collectionIds: {
-        products: env.WEBFLOW_COLLECTION_PRODUCTS_ID,
-        skus: env.WEBFLOW_COLLECTION_SKUS_ID,
-    },
-    token: env.WEBFLOW_AUTH,
-    webhookSecret: env.WEBFLOW_WEBHOOK_SECRET,
-});

apps/portal/src/routes/(app)/apparel/orders/+page.svelte

@@ -6,7 +6,7 @@
         Awaited<ReturnType<typeof api.commerce.orders.get>>["data"]
     >[number];
 
-    const LIMIT = 25;
+    const LIMIT = 5;
 
     let orders = $state<OrderRow[]>([]);
     let loading = $state(true);

apps/portal/src/routes/(app)/apparel/products/+page.server.ts

@@ -1,12 +0,0 @@
-import type { PageServerLoad } from "./$types";
-import { printful } from "$lib/printful";
-import { webflow } from "$lib/webflow";
-
-export const load = async ({ }: Parameters<PageServerLoad>[0]) => {
-    return {
-        products: {
-            printful: printful.Products.list({ forceAll: true }),
-            webflow: webflow.Products.list({ forceAll: true }),
-        },
-    };
-};

apps/portal/src/routes/(app)/apparel/products/+page.svelte

@@ -1,13 +1,58 @@
 <script lang="ts">
-    import type { Printful } from "@blade-and-brawn/commerce";
     import { onMount } from "svelte";
     import { api } from "$lib/api.js";
     import type { EventStatus } from "@blade-and-brawn/api";
 
-    const { data } = $props();
+    type ProductRow = NonNullable<
+        Awaited<ReturnType<typeof api.commerce.products.get>>["data"]
+    >[number];
 
     type SyncStatus = EventStatus | "none";
 
+    const LIMIT = 25;
+
+    let products = $state<ProductRow[]>([]);
+    let loading = $state(true);
+    let loadError = $state<string | null>(null);
+    let offset = $state(0);
+
+    function errorMessage(err: unknown): string {
+        const value = (err as { value?: { error?: string } })?.value;
+        return (
+            value?.error ??
+            (err instanceof Error ? err.message : "Unknown error")
+        );
+    }
+
+    async function refresh() {
+        loading = true;
+        loadError = null;
+        try {
+            const res = await api.commerce.products.get({
+                query: {
+                    limit: LIMIT,
+                    offset,
+                },
+            });
+            if (res.error) throw res.error;
+            products = res.data as ProductRow[];
+        } catch (err) {
+            loadError = errorMessage(err);
+        } finally {
+            loading = false;
+        }
+    }
+
+    function prevPage() {
+        offset = Math.max(0, offset - LIMIT);
+        refresh();
+    }
+
+    function nextPage() {
+        offset += LIMIT;
+        refresh();
+    }
+
     const synchronizer = $state({
         status: "none" as SyncStatus,
         syncingPProductIds: [] as number[],
@@ -30,8 +75,12 @@
         },
         startPolling: () => {
             const intervalId = setInterval(async () => {
+                const wasInProgress = synchronizer.isInProgress;
                 await synchronizer.poll();
-                if (!synchronizer.isInProgress) clearInterval(intervalId);
+                if (!synchronizer.isInProgress) {
+                    clearInterval(intervalId);
+                    if (wasInProgress) refresh();
+                }
             }, 100);
         },
         syncAll: async () => {
@@ -49,23 +98,15 @@
         if (synchronizer.isInProgress) {
             synchronizer.startPolling();
         }
+        await refresh();
     });
-
-    const isProductSynced = async (pProduct: Printful.Products.SyncProduct) => {
-        const wProducts = await data.products.webflow;
-        return wProducts.some(
-            (p) => p.product.id === pProduct.external_id.split("-")[0],
-        );
-    };
 </script>
 
-{#await data.products.printful}
-    <p>Loading...</p>
-{:then pProducts}
+<div class="w-full flex items-center gap-2 flex-wrap mt-10 mb-5">
     <button
         disabled={synchronizer.isInProgress}
         onclick={() => synchronizer.syncAll()}
-        class="btn btn-xl mb-5 mt-10"
+        class="btn btn-xl"
     >
         {#if synchronizer.isInProgress}
             Syncing...
@@ -73,64 +114,89 @@
             Sync all
         {/if}
     </button>
-    <div class="w-full overflow-x-auto">
-        <table class="table">
-            <thead>
-                <tr>
-                    <th>Name</th>
-                    <th>Status</th>
-                    <th>Sync</th>
-                </tr>
-            </thead>
-            <tbody>
-                {#each pProducts as pProduct}
-                    <tr class="hover:bg-base-300">
-                        <td
-                            ><button
-                                onclick={async () => {
-                                    const res = await api.commerce
-                                        .products({
-                                            pProductId: pProduct.id,
-                                        })
-                                        .get();
-                                    console.log(res.data);
-                                }}
-                                class="cursor-pointer underline"
-                            >
-                                {pProduct.name}
-                            </button></td
-                        >
-                        <td>
-                            {#await isProductSynced(pProduct) then isSynced}
-                                {#if synchronizer.syncingPProductIds.includes(pProduct.id)}
-                                    <div class="badge badge-warning">
-                                        Syncing...
-                                    </div>
-                                {:else if isSynced}
-                                    <div class="badge badge-success">
-                                        Synced
-                                    </div>
-                                {:else}
-                                    <div class="badge badge-error">
-                                        Desynced
-                                    </div>
-                                {/if}
-                            {/await}
-                        </td>
-                        <td>
-                            <button
-                                disabled={synchronizer.status === "processing"}
-                                onclick={() => synchronizer.sync(pProduct.id)}
-                                class="btn"
-                            >
-                                Sync
-                            </button>
-                        </td>
-                    </tr>
-                {/each}
-            </tbody>
-        </table>
+
+    <button class="btn btn-sm" onclick={refresh} disabled={loading}>
+        {loading ? "Refreshing..." : "Refresh"}
+    </button>
+
+    <div class="flex-1"></div>
+
+    <button
+        class="btn btn-sm"
+        onclick={prevPage}
+        disabled={offset === 0 || loading}
+    >
+        Prev
+    </button>
+    <button
+        class="btn btn-sm"
+        onclick={nextPage}
+        disabled={products.length < LIMIT || loading}
+    >
+        Next
+    </button>
+</div>
+
+{#if loadError}
+    <div role="alert" class="alert alert-error w-full mb-4">
+        <span>{loadError}</span>
     </div>
-{:catch error}
-    <p>Something went wrong: {error.message}</p>
-{/await}
+{/if}
+
+<div class="w-full overflow-x-auto">
+    <table class="table">
+        <thead>
+            <tr>
+                <th>Name</th>
+                <th>Status</th>
+                <th>Sync</th>
+            </tr>
+        </thead>
+        <tbody>
+            {#each products as product (product.pProduct.id)}
+                <tr class="hover:bg-base-300">
+                    <td
+                        ><button
+                            onclick={async () => {
+                                const res = await api.commerce
+                                    .products({
+                                        pProductId: product.pProduct.id,
+                                    })
+                                    .get();
+                                console.log(res.data);
+                            }}
+                            class="cursor-pointer underline"
+                        >
+                            {product.pProduct.name}
+                        </button></td
+                    >
+                    <td>
+                        {#if synchronizer.syncingPProductIds.includes(product.pProduct.id)}
+                            <div class="badge badge-warning">Syncing...</div>
+                        {:else if product.isSynced}
+                            <div class="badge badge-success">Synced</div>
+                        {:else}
+                            <div class="badge badge-error">Desynced</div>
+                        {/if}
+                    </td>
+                    <td>
+                        <button
+                            disabled={synchronizer.status === "processing"}
+                            onclick={() =>
+                                synchronizer.sync(product.pProduct.id)}
+                            class="btn"
+                        >
+                            Sync
+                        </button>
+                    </td>
+                </tr>
+            {:else}
+                <tr>
+                    <td colspan="3" class="text-center opacity-60">
+                        {loading ? "Loading..." : "No products found"}
+                    </td>
+                </tr>
+            {/each}
+        </tbody>
+    </table>
+</div>