Merge pull request #2096 from OneCommunityGlobal/venkataramanan_fix_team_code_reflect_from_weekly_summaries

Venkataramanan fix: quick setup team code fix

Author: one-community

src/controllers/reportsController.js

@@ -730,6 +730,30 @@ const reportsController = function () {
     }
   };
 
+  const getAllDistinctTeamCodes = async (req, res) => {
+    const requestor = safeRequestorFromReq(req);
+
+    try {
+      const allowed = await hasPermission(requestor, 'getWeeklySummaries');
+      if (!allowed) {
+        return res.status(403).send('You are not authorized to view team codes');
+      }
+
+      const teamCodes = await UserProfile.distinct('teamCode', {
+        teamCode: { $nin: [null, ''] },
+      });
+
+      const sortedTeamCodes = teamCodes.sort((a, b) => a.localeCompare(b));
+
+      return res.status(200).send(sortedTeamCodes);
+    } catch (error) {
+      console.error('Error fetching distinct team codes:', error);
+      return res.status(500).send({
+        error: 'An error occurred while fetching team codes.',
+      });
+    }
+  };
+
   const getReportTeamCodes = async (req, res) => {
     try {
       // const minActive = Number(req.query.activeMembersMinimum ?? 1);
@@ -843,6 +867,7 @@ const reportsController = function () {
     invalidateWeeklySummariesCache,
     getAdminList,
     sendEmailReport,
+    getAllDistinctTeamCodes,
   };
 };
 

src/controllers/userProfileController.js

@@ -2714,10 +2714,8 @@ const createControllerMethods = function (UserProfile, Project, cache) {
                 },
               },
             },
-            userInfo: {
-              userId: user._id,
-              teamCodeWarning,
-            },
+            userId: user._id.toString(),
+            teamCodeWarning,
           };
         }),
       );
@@ -2732,7 +2730,10 @@ const createControllerMethods = function (UserProfile, Project, cache) {
 
       return res.status(200).send({
         message: 'Team codes updated successfully.',
-        updatedUsers: updatedUsersInfo,
+        updatedUsers: updatedUsersInfo.map(({ userId, teamCodeWarning }) => ({
+          userId,
+          teamCodeWarning,
+        })),
       });
     } catch (error) {
       console.error('Error updating team codes:', error);

src/routes/reportsRouter.js

@@ -14,6 +14,7 @@ const route = function () {
   reportsRouter.route('/reports/getrecepients').get(controller.getReportRecipients);
 
   reportsRouter.route('/reports/weeklysummaries').get(controller.getWeeklySummaries);
+  reportsRouter.get('/reports/weeklysummaries/teamcodes', controller.getAllDistinctTeamCodes);
 
   reportsRouter
     .route('/reports/overviewsummaries/volunteerstats')