Add arguments

src/datasources/product-api.ts

@@ -6,6 +6,7 @@ import {
   PrintProduct,
   ProductBlacklist,
 } from './product-types';
+import { Maybe } from './types';
 
 const MINUTE = 60;
 
@@ -34,17 +35,22 @@ export class ProductAPI extends SQLDataSource {
     });
   }
 
-  async getProducts(args: any): Promise<Product> {
-    const query = sql.products(true, true, 10);
-    // console.log(query);
-    const data = await this.knex.raw(query).then((data) => {
-      // console.log(data);
-      return data.rows.map((row) => {
+  async getProducts(
+    limit: Maybe<number>,
+    visible: Maybe<boolean>,
+    browsable: Maybe<boolean>,
+  ): Promise<Product> {
+    // console.log(limit, browsable, visible);
+    limit = limit ?? 100;
+    const query = sql.products(limit, visible, browsable);
+
+    const data = await this.knex.raw(query).then((data) =>
+      data.rows.map((row) => {
         row.blacklisting = this.getBlacklisting(row);
         row.printProducts = this.getPrintProducts(row);
         return row;
-      });
-    });
+      }),
+    );
 
     // console.log(JSON.stringify(data, null, 2));
 

src/datasources/sql/products-sql.ts

@@ -1,9 +1,11 @@
+import { Maybe } from '../types';
+
 // Get syntax highlighting with vscode by installing "Comment tagged templates2
 
 export function products(
-  visible: boolean | null,
-  browsable: boolean | null,
   limit: number = 100,
+  visible: Maybe<boolean>,
+  browsable: Maybe<boolean>,
 ) {
   const query = /* sql */ `
 
@@ -55,7 +57,6 @@ export function products(
   AND products.browsable = ${browsable ? '1' : '0'}`
       : ``
   }
-GROUP BY products.productid
 LIMIT ${limit}
 `;
 

src/datasources/types.ts

@@ -0,0 +1 @@
+export type Maybe<T> = T | undefined;