Merge pull request #78 from topcoder-platform/develop

push to prod

Author: ThomasKranitsas

src/models/Challenge.js

@@ -14,9 +14,7 @@ const schema = new Schema({
   },
   legacyId: {
     type: Number,
-    required: false,
-    rangeKey: true,
-    index: true
+    required: false
   },
   typeId: {
     type: String,
@@ -122,8 +120,7 @@ const schema = new Schema({
     required: false
   }
 },
-{
-  throughput: { read: 4, write: 2 }
-})
+{ throughput: 'ON_DEMAND' }
+)
 
 module.exports = schema

src/services/ChallengeService.js

@@ -29,7 +29,7 @@ async function filterChallengesByGroupsAccess (currentUser, challenges) {
   const needToCheckForGroupAccess = !currentUser ? true : !currentUser.isMachine && !helper.hasAdminRole(currentUser)
   console.log('needToCheckForGroupAccess', needToCheckForGroupAccess)
   for (const challenge of challenges) {
-    challenge.groups = _.filter(challenge.groups, g => _.toString(g).toLowerCase() !== 'null')
+    challenge.groups = _.filter(challenge.groups, g => !_.includes(['null', 'undefined'], _.toString(g).toLowerCase()))
     console.log('challenge.groups', challenge.groups)
     if (!challenge.groups || _.get(challenge, 'groups.length', 0) === 0 || !needToCheckForGroupAccess) {
       res.push(challenge)