Merge pull request #320 from topcoder-platform/feature/filter_disabled_templates_from_list_endpoint

stopping disabled templates to be returned in response of list endpoints

Author: RishiRajSahu

src/routes/metadata/list.js

@@ -27,6 +27,10 @@ function getUsedModel() {
     priceConfig: { },
   };
   const query = {
+    where: {
+      deletedAt: { $eq: null },
+      disabled: false,
+    },
     attributes: { exclude: ['deletedAt', 'deletedBy'] },
     raw: true,
   };

src/routes/productTemplates/list.js

@@ -14,7 +14,7 @@ module.exports = [
     if (!util.isValidFilter(filters, ['productKey'])) {
       return util.handleError('Invalid filters', null, req, next);
     }
-    const where = { deletedAt: { $eq: null } };
+    const where = { deletedAt: { $eq: null }, disabled: false };
     if (filters.productKey) {
       where.productKey = { $eq: filters.productKey };
     }

src/routes/productTemplates/list.spec.js

@@ -51,7 +51,7 @@ describe('LIST product templates', () => {
         },
         alias2: [1, 2, 3],
       },
-      disabled: true,
+      disabled: false,
       hidden: true,
       isAddOn: true,
       template: {

src/routes/projectTemplates/list.js

@@ -12,6 +12,7 @@ module.exports = [
   (req, res, next) => models.ProjectTemplate.findAll({
     where: {
       deletedAt: { $eq: null },
+      disabled: false,
     },
     attributes: { exclude: ['deletedAt', 'deletedBy'] },
     raw: true,

src/routes/projectTemplates/list.spec.js

@@ -20,7 +20,7 @@ describe('LIST project templates', () => {
       question: 'question 1',
       info: 'info 1',
       aliases: ['key-1', 'key_1'],
-      disabled: true,
+      disabled: false,
       hidden: true,
       scope: {
         scope1: {