Fix(account-transactions): Include new GraphQL args in query

opencollective · Apr 8, 2024 · e41b4fb · e41b4fb
1 parent aeb06a2
commit e41b4fb
Showing 1 changed file with 22 additions and 13 deletions.
diff --git a/src/server/controllers/account-transactions.js b/src/server/controllers/account-transactions.js
@@ -206,6 +206,10 @@ const transactionsQuery = gqlV2/* GraphQL */ `
       includeGiftCardTransactions: $includeGiftCardTransactions
       includeRegularTransactions: $includeRegularTransactions
       paymentMethodType: $paymentMethodType
+      expenseType: $expenseType
+      expense: $expense
+      order: $order
+      isRefund: $isRefund
     ) {
       ...TransactionsFragment
     }
@@ -259,6 +263,10 @@ const hostTransactionsQuery = gqlV2/* GraphQL */ `
       includeChildrenTransactions: $includeChildrenTransactions
       includeHost: $includeHost
       paymentMethodType: $paymentMethodType
+      expenseType: $expenseType
+      expense: $expense
+      order: $order
+      isRefund: $isRefund
     ) {
       ...TransactionsFragment
     }
@@ -436,6 +444,7 @@ const accountTransactions = async (req, res) => {
     return res.status(405).send({ error: { message: 'Method not allowed' } });
   }
 
+  let newVars = {};
   const variables = pick({ ...req.params, ...req.query }, [
     'account',
     'slug',
@@ -557,22 +566,22 @@ const accountTransactions = async (req, res) => {
     variables.kind = difference(variables.kind || allKinds, ['TAX']);
   }
 
-  if (variables.expenseType) {
-    variables.expenseType = variables.expenseType.split(',').map(toUpper).map(trim);
-  }
+  // if (variables.expenseType) {
+  //   variables.expenseType = variables.expenseType.split(',').map(toUpper).map(trim);
+  // }
 
-  if (variables.orderId) {
-    variables.order = { legacyId: parseInt(variables.orderId) };
-  }
+  // if (variables.orderId) {
+  //   variables.order = { legacyId: parseInt(variables.orderId) };
+  // }
 
-  if (variables.expenseId) {
-    variables.expense = { legacyId: parseInt(variables.expenseId) };
-  }
+  // if (variables.expenseId) {
+  //   variables.expense = { legacyId: parseInt(variables.expenseId) };
+  // }
 
-  // isRefund can be false but default should be undefined
-  if (!isNil(variables.isRefund)) {
-    variables.isRefund = parseToBooleanDefaultFalse(variables.isRefund);
-  }
+  // // isRefund can be false but default should be undefined
+  // if (!isNil(variables.isRefund)) {
+  //   variables.isRefund = parseToBooleanDefaultFalse(variables.isRefund);
+  // }
 
   if (req.query.fullDescription) {
     variables.fullDescription = parseToBooleanDefaultFalse(req.query.fullDescription);