Improve product image handling and add costPerUnit

Added a utility to generate product image URLs, ensuring images are displayed correctly in the product table. Updated the Product model to include an optional costPerUnit field. Minor UI and code formatting improvements were made for consistency.

app/dashboard/products/page.tsx

@@ -45,7 +45,7 @@ const ProfitAnalysisModal = dynamic(() => import("@/components/modals/profit-ana
 
 function ProductTableSkeleton() {
   return (
-    <Card className="animate-in fade-in duration-500">
+    <Card className="animate-in fade-in duration-500 border-border/40 bg-background/50 backdrop-blur-sm shadow-sm">
       <CardHeader>
         <div className="flex items-center justify-between">
           <Skeleton className="h-6 w-32" />
@@ -152,7 +152,7 @@ export default function ProductsPage() {
   const [importModalOpen, setImportModalOpen] = useState(false);
   const [addProductOpen, setAddProductOpen] = useState(false);
   const [profitAnalysisOpen, setProfitAnalysisOpen] = useState(false);
-  const [selectedProductForAnalysis, setSelectedProductForAnalysis] = useState<{id: string, name: string} | null>(null);
+  const [selectedProductForAnalysis, setSelectedProductForAnalysis] = useState<{ id: string, name: string } | null>(null);
 
   // Fetch products and categories
   useEffect(() => {
@@ -210,7 +210,7 @@ export default function ProductsPage() {
     }));
   };
 
-    // Handle input changes
+  // Handle input changes
   const handleChange = (
     e: ChangeEvent<HTMLInputElement | HTMLTextAreaElement>
   ) => setProductData({ ...productData, [e.target.name]: e.target.value });
@@ -323,9 +323,9 @@ export default function ProductsPage() {
       ...product,
       pricing: product.pricing
         ? product.pricing.map((tier) => ({
-            minQuantity: tier.minQuantity,
+          minQuantity: tier.minQuantity,
-            pricePerUnit: tier.pricePerUnit,
+          pricePerUnit: tier.pricePerUnit,
-          }))
+        }))
         : [{ minQuantity: 1, pricePerUnit: 0 }],
       costPerUnit: product.costPerUnit || 0,
     });
@@ -343,9 +343,9 @@ export default function ProductsPage() {
       image: null, // Clear image so user can upload a new one
       pricing: product.pricing
         ? product.pricing.map((tier) => ({
-            minQuantity: tier.minQuantity,
+          minQuantity: tier.minQuantity,
-            pricePerUnit: tier.pricePerUnit,
+          pricePerUnit: tier.pricePerUnit,
-          }))
+        }))
         : [{ minQuantity: 1, pricePerUnit: 0 }],
       costPerUnit: product.costPerUnit || 0,
       // Reset stock to defaults for cloned product

components/tables/product-table.tsx

@@ -14,11 +14,22 @@ import {
   AlertCircle,
   Calculator,
   Copy,
+  PackageX,
+  Archive
 } from "lucide-react";
 import { Button } from "@/components/ui/button";
 import { Product } from "@/models/products";
 import { Badge } from "@/components/ui/badge";
 import { Switch } from "@/components/ui/switch";
+import { Card, CardContent, CardHeader, CardTitle } from "@/components/ui/card";
+import { motion, AnimatePresence } from "framer-motion";
+
+const getProductImageUrl = (product: Product) => {
+  if (!product.image) return null;
+  if (typeof product.image === 'string' && product.image.startsWith('http')) return product.image;
+  // Use the API endpoint to serve the image
+  return `${process.env.NEXT_PUBLIC_API_URL}/products/${product._id}/image`;
+};
 
 interface ProductTableProps {
   products: Product[];
@@ -68,35 +79,57 @@ const ProductTable = ({
     }
   };
 
-  const renderProductRow = (product: Product, isDisabled: boolean = false) => (
+  const renderProductRow = (product: Product, index: number, isDisabled: boolean = false) => (
-    <TableRow
+    <motion.tr
       key={product._id}
-      className={`transition-colors hover:bg-gray-50 dark:hover:bg-zinc-800/70 ${isDisabled ? "opacity-60" : ""}`}
+      initial={{ opacity: 0, y: 10 }}
+      animate={{ opacity: 1, y: 0 }}
+      exit={{ opacity: 0 }}
+      transition={{ duration: 0.2, delay: index * 0.05 }}
+      className={`group hover:bg-muted/40 border-b border-border/50 transition-colors ${isDisabled ? "opacity-60 bg-muted/20" : ""}`}
     >
-      <TableCell>
+      <TableCell className="font-medium">
-        <div className="font-medium truncate max-w-[180px]">{product.name}</div>
+        <div className="flex items-center gap-2">
-        <div className="hidden sm:block text-sm text-muted-foreground mt-1">
+          <div className="h-8 w-8 rounded bg-muted/50 flex items-center justify-center text-muted-foreground overflow-hidden relative">
-          {getCategoryNameById(product.category)}
+            {getProductImageUrl(product) ? (
+              <img
+                src={getProductImageUrl(product)!}
+                alt={product.name}
+                className="h-full w-full object-cover"
+              />
+            ) : (
+              <span className="text-xs font-bold">{product.name.charAt(0).toUpperCase()}</span>
+            )}
+          </div>
+          <div>
+            <div className="truncate max-w-[180px]">{product.name}</div>
+            <div className="sm:hidden text-xs text-muted-foreground">
+              {getCategoryNameById(product.category)}
+            </div>
+          </div>
         </div>
       </TableCell>
       <TableCell className="hidden sm:table-cell text-center">
-        {getCategoryNameById(product.category)}
+        <Badge variant="outline" className="font-normal bg-background/50">
+          {getCategoryNameById(product.category)}
+        </Badge>
       </TableCell>
-      <TableCell className="hidden md:table-cell text-center">
+      <TableCell className="hidden md:table-cell text-center text-muted-foreground text-sm">
         {product.unitType}
       </TableCell>
       <TableCell className="text-center">
         {product.stockTracking ? (
-          <div className="flex items-center justify-center gap-1">
+          <div className="flex items-center justify-center gap-1.5">
             {getStockIcon(product)}
-            <span className="text-sm">
+            <span className={`text-sm font-medium ${product.stockStatus === 'out_of_stock' ? 'text-destructive' :
-              {product.currentStock !== undefined ? product.currentStock : 0}{" "}
+              product.stockStatus === 'low_stock' ? 'text-amber-500' : 'text-foreground'
-              {product.unitType}
+              }`}>
+              {product.currentStock !== undefined ? product.currentStock : 0}
             </span>
           </div>
         ) : (
-          <Badge variant="outline" className="text-xs">
+          <Badge variant="secondary" className="text-[10px] h-5 px-1.5 text-muted-foreground bg-muted/50">
-            Not Tracked
+            Unlimited
           </Badge>
         )}
       </TableCell>
@@ -106,57 +139,61 @@ const ProductTable = ({
           onCheckedChange={(checked) =>
             onToggleEnabled(product._id as string, checked)
           }
+          className="data-[state=checked]:bg-primary"
         />
       </TableCell>
-      <TableCell className="text-right flex justify-end space-x-1">
+      <TableCell className="text-right">
-        {onProfitAnalysis && (
+        <div className="flex items-center justify-end gap-1">
+          {onProfitAnalysis && (
+            <Button
+              variant="ghost"
+              size="icon"
+              onClick={() =>
+                onProfitAnalysis(product._id as string, product.name)
+              }
+              className="h-8 w-8 text-emerald-500 hover:text-emerald-600 hover:bg-emerald-500/10"
+              title="Profit Analysis"
+            >
+              <Calculator className="h-4 w-4" />
+            </Button>
+          )}
+          {onClone && (
+            <Button
+              variant="ghost"
+              size="icon"
+              onClick={() => onClone(product)}
+              className="h-8 w-8 text-blue-500 hover:text-blue-600 hover:bg-blue-500/10"
+              title="Clone Listing"
+            >
+              <Copy className="h-4 w-4" />
+            </Button>
+          )}
           <Button
             variant="ghost"
-            size="sm"
+            size="icon"
-            onClick={() =>
+            onClick={() => onEdit(product)}
-              onProfitAnalysis(product._id as string, product.name)
+            className="h-8 w-8 text-muted-foreground hover:text-foreground"
-            }
+            title="Edit Product"
-            className="text-green-600 hover:text-green-700 hover:bg-green-50 dark:hover:bg-green-950/20"
-            title="Profit Analysis"
           >
-            <Calculator className="h-4 w-4" />
+            <Edit className="h-4 w-4" />
           </Button>
-        )}
-        {onClone && (
           <Button
             variant="ghost"
-            size="sm"
+            size="icon"
-            onClick={() => onClone(product)}
+            onClick={() => onDelete(product._id as string)}
-            className="text-blue-600 hover:text-blue-700 hover:bg-blue-50 dark:hover:bg-blue-950/20"
+            className="h-8 w-8 text-muted-foreground hover:text-destructive hover:bg-destructive/10"
-            title="Clone Listing"
+            title="Delete Product"
           >
-            <Copy className="h-4 w-4" />
+            <Trash className="h-4 w-4" />
           </Button>
-        )}
+        </div>
-        <Button
-          variant="ghost"
-          size="sm"
-          onClick={() => onEdit(product)}
-          title="Edit Product"
-        >
-          <Edit className="h-4 w-4" />
-        </Button>
-        <Button
-          variant="ghost"
-          size="sm"
-          onClick={() => onDelete(product._id as string)}
-          className="text-red-500 hover:text-red-600 hover:bg-red-50 dark:hover:bg-red-950/20"
-          title="Delete Product"
-        >
-          <Trash className="h-4 w-4" />
-        </Button>
       </TableCell>
-    </TableRow>
+    </motion.tr>
   );
 
   const renderTableHeader = () => (
-    <TableHeader className="bg-gray-50 dark:bg-zinc-800/50">
+    <TableHeader className="bg-muted/50 sticky top-0 z-10">
-      <TableRow className="hover:bg-transparent">
+      <TableRow className="hover:bg-transparent border-border/50">
         <TableHead className="w-[200px]">Product</TableHead>
         <TableHead className="hidden sm:table-cell text-center">
           Category
@@ -166,57 +203,86 @@ const ProductTable = ({
         <TableHead className="hidden lg:table-cell text-center">
           Enabled
         </TableHead>
-        <TableHead className="text-right">Actions</TableHead>
+        <TableHead className="text-right pr-6">Actions</TableHead>
       </TableRow>
     </TableHeader>
   );
 
   return (
-    <div className="space-y-6">
+    <div className="space-y-8">
       {/* Enabled Products Table */}
-      <div className="rounded-lg border dark:border-zinc-700 shadow-sm overflow-hidden">
+      <Card className="border-border/40 bg-background/50 backdrop-blur-sm shadow-sm overflow-hidden">
-        <Table className="relative">
+        <CardHeader className="py-4 px-6 border-b border-border/50 bg-muted/30">
-          {renderTableHeader()}
+          <CardTitle className="text-lg font-medium flex items-center gap-2">
-          <TableBody>
+            <CheckCircle className="h-5 w-5 text-primary" />
-            {loading ? (
+            Active Products
-              Array.from({ length: 1 }).map((_, index) => (
+            <Badge variant="secondary" className="ml-2 bg-background/80 backdrop-blur-sm">
-                <TableRow key={index}>
+              {sortedEnabledProducts.length}
-                  <TableCell>Loading...</TableCell>
+            </Badge>
-                  <TableCell>Loading...</TableCell>
+          </CardTitle>
-                  <TableCell>Loading...</TableCell>
+        </CardHeader>
-                  <TableCell>Loading...</TableCell>
+        <CardContent className="p-0">
-                  <TableCell>Loading...</TableCell>
+          <div className="max-h-[600px] overflow-auto">
-                  <TableCell>Loading...</TableCell>
+            <Table>
-                </TableRow>
+              {renderTableHeader()}
-              ))
+              <TableBody>
-            ) : sortedEnabledProducts.length > 0 ? (
+                <AnimatePresence mode="popLayout">
-              sortedEnabledProducts.map((product) => renderProductRow(product))
+                  {loading ? (
-            ) : (
+                    <TableRow>
-              <TableRow>
+                      <TableCell colSpan={6} className="h-32 text-center text-muted-foreground">
-                <TableCell colSpan={6} className="h-24 text-center">
+                        Loading products...
-                  No enabled products found.
+                      </TableCell>
-                </TableCell>
+                    </TableRow>
-              </TableRow>
+                  ) : sortedEnabledProducts.length > 0 ? (
-            )}
+                    sortedEnabledProducts.map((product, index) => renderProductRow(product, index))
-          </TableBody>
+                  ) : (
-        </Table>
+                    <TableRow>
-      </div>
+                      <TableCell colSpan={6} className="h-32 text-center text-muted-foreground">
+                        <div className="flex flex-col items-center justify-center gap-2">
+                          <PackageX className="h-8 w-8 opacity-50" />
+                          <p>No active products found</p>
+                        </div>
+                      </TableCell>
+                    </TableRow>
+                  )}
+                </AnimatePresence>
+              </TableBody>
+            </Table>
+          </div>
+        </CardContent>
+      </Card>
 
       {/* Disabled Products Section */}
       {!loading && disabledProducts.length > 0 && (
-        <div className="rounded-lg border dark:border-zinc-700 shadow-sm overflow-hidden bg-gray-50/30 dark:bg-zinc-900/30">
+        <Card className="border-border/40 bg-background/30 backdrop-blur-sm shadow-sm overflow-hidden opacity-90">
-          <Table className="relative">
+          <CardHeader className="py-4 px-6 border-b border-border/50 bg-muted/20">
-            {renderTableHeader()}
+            <CardTitle className="text-lg font-medium flex items-center gap-2 text-muted-foreground">
-            <TableBody>
+              <Archive className="h-5 w-5" />
-              {sortedDisabledProducts.map((product) =>
+              Archived / Disabled
-                renderProductRow(product, true),
+              <Badge variant="outline" className="ml-2">
-              )}
+                {sortedDisabledProducts.length}
-            </TableBody>
+              </Badge>
-          </Table>
+            </CardTitle>
-        </div>
+          </CardHeader>
+          <CardContent className="p-0">
+            <div className="max-h-[400px] overflow-auto">
+              <Table>
+                {renderTableHeader()}
+                <TableBody>
+                  <AnimatePresence mode="popLayout">
+                    {sortedDisabledProducts.map((product, index) =>
+                      renderProductRow(product, index, true),
+                    )}
+                  </AnimatePresence>
+                </TableBody>
+              </Table>
+            </div>
+          </CardContent>
+        </Card>
       )}
     </div>
   );
+
 };
 
 export default ProductTable;

models/products.ts

@@ -15,4 +15,5 @@ export interface Product {
     pricePerUnit: number;
   }>;
   image?: string | File | null | undefined;
+  costPerUnit?: number;
 }