model: snapshot to fetch members with role:member

Author: harshith-venkatesh

controllers/members.js

@@ -1,4 +1,4 @@
-const { fetchMembers, migrateUsers, deleteIsMemberProperty } = require('../models/members')
+const { fetchMembers, migrateUsers, deleteIsMemberProperty, fetchOnlyMembers } = require('../models/members')
 const tasks = require('../models/tasks')
 
 /**
@@ -31,9 +31,9 @@ const getMembers = async (req, res) => {
 
 const getIdleMembers = async (req, res) => {
   try {
-    const allMembers = await fetchMembers()
+    const onlyMembers = await fetchOnlyMembers()
     const taskParticipants = await tasks.fetchActiveTaskMembers()
-    const idleMembers = allMembers?.filter(({ id }) => !taskParticipants.has(id))
+    const idleMembers = onlyMembers?.filter(({ id }) => !taskParticipants.has(id))
     const idleMemberUserNames = idleMembers?.map((member) => member.username)
 
     return res.json({

models/members.js

@@ -98,8 +98,37 @@ const deleteIsMemberProperty = async () => {
   }
 }
 
+/**
+ * Fetches the data about our members with role member:true
+ * @return {Promise<userModel|Array>}
+ */
+
+const fetchOnlyMembers = async () => {
+  try {
+    const snapshot = await userModel.where('roles.member', '==', true).get()
+    const onlyMembers = []
+
+    if (!snapshot.empty) {
+      snapshot.forEach((doc) => {
+        onlyMembers.push({
+          id: doc.id,
+          ...doc.data(),
+          phone: undefined,
+          email: undefined,
+          tokens: undefined
+        })
+      })
+    }
+    return onlyMembers
+  } catch (err) {
+    logger.error('Error retrieving members data with roles of member', err)
+    throw err
+  }
+}
+
 module.exports = {
   fetchMembers,
   migrateUsers,
-  deleteIsMemberProperty
+  deleteIsMemberProperty,
+  fetchOnlyMembers
 }