Skip to content

Commit 4592e15

Browse files
add filter queries
1 parent f8e7a16 commit 4592e15

File tree

2 files changed

+13
-5
lines changed

2 files changed

+13
-5
lines changed

controllers/users.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -496,8 +496,8 @@ const filterUsers = async (req, res) => {
496496
};
497497

498498
const returnAllUsers = async (req, res) => {
499-
const data = await userQuery.getAllUsers();
500-
return data;
499+
const data = await userQuery.getDiscordUsers();
500+
return res.json(data);
501501
};
502502

503503
module.exports = {

models/users.js

Lines changed: 11 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -456,10 +456,18 @@ const getUsersBasedOnFilter = async (query) => {
456456
* @return {Promise<users>}
457457
*/
458458

459-
const getAllUsers = async () => {
459+
const getDiscordUsers = async () => {
460460
try {
461461
const usersRef = await userModel.where("roles.archived", "==", false).get();
462-
return usersRef;
462+
const users = [];
463+
usersRef.forEach((user) => {
464+
if (user.data()?.discordId && user.data().roles?.in_discord === false)
465+
users.push({
466+
id: user.id,
467+
...user.data(),
468+
});
469+
});
470+
return users;
463471
} catch (err) {
464472
logger.error(`Error while fetching all users: ${err}`);
465473
throw err;
@@ -481,5 +489,5 @@ module.exports = {
481489
getRdsUserInfoByGitHubUsername,
482490
fetchUsers,
483491
getUsersBasedOnFilter,
484-
getAllUsers,
492+
getDiscordUsers,
485493
};

0 commit comments

Comments
 (0)