Fixed membership request being added in the user's DB

eslint.config.mjs

@@ -14,8 +14,8 @@ import * as graphqlEslint from "@graphql-eslint/eslint-plugin";
 const __filename = fileURLToPath(import.meta.url);
 const __dirname = path.dirname(__filename);
 const compat = new FlatCompat({
-  baseDirectory: __dirname,
-  recommendedConfig: js.configs.recommended,
+    baseDirectory: __dirname,
+    recommendedConfig: js.configs.recommended
 });
 
 export default [
@@ -44,7 +44,9 @@ export default [
 
     languageOptions: {
       parser: tsParser,
-      globals: globals.node,
+      globals: {
+        ...globals.node,
+      },
     },
     rules: {
       "no-restricted-imports": [
@@ -142,7 +144,7 @@ export default [
     },
 
     languageOptions: {
-      parser: parser,
+      parser: graphqlEslint.parser,
     },
   },
   {

src/resolvers/Mutation/sendMembershipRequest.ts

@@ -78,8 +78,9 @@ export const sendMembershipRequest: MutationResolvers["sendMembershipRequest"] =
 
     // Checks if the user is blocked
     const user = await User.findById(context.userId).lean();
+
     if (
-      user !== null &&
+      user != null &&
       organization.blockedUsers.some((blockedUser) =>
         new mongoose.Types.ObjectId(blockedUser.toString()).equals(user._id),
       )
@@ -96,21 +97,38 @@ export const sendMembershipRequest: MutationResolvers["sendMembershipRequest"] =
       user: context.userId,
       organization: organization._id,
     });
-
     if (membershipRequestExists) {
+      // Check if the request is already in the user's document
+      if (
+        user != null &&
+        !user.membershipRequests.includes(membershipRequestExists._id)
+      ) {
+        // If it's not in the user's document, add it
+        await User.findByIdAndUpdate(
+          context.userId,
+          {
+            $push: {
+              membershipRequests: membershipRequestExists._id,
+            },
+          },
+          { new: true, runValidators: true },
+        );
+      }
+
       throw new errors.ConflictError(
         requestContext.translate(MEMBERSHIP_REQUEST_ALREADY_EXISTS.MESSAGE),
         MEMBERSHIP_REQUEST_ALREADY_EXISTS.CODE,
         MEMBERSHIP_REQUEST_ALREADY_EXISTS.PARAM,
       );
     }
 
+    // Creating Membership Request
     const createdMembershipRequest = await MembershipRequest.create({
       user: context.userId,
       organization: organization._id,
     });
 
-    // add membership request to organization
+    // Updating Membership Request in organization
     const updatedOrganization = await Organization.findOneAndUpdate(
       {
         _id: organization._id,
@@ -129,16 +147,15 @@ export const sendMembershipRequest: MutationResolvers["sendMembershipRequest"] =
       await cacheOrganizations([updatedOrganization]);
     }
 
-    // add membership request to user
-    await User.updateOne(
-      {
-        _id: context.userId,
-      },
+    // Updating User
+    await User.findByIdAndUpdate(
+      context.userId,
       {
         $push: {
           membershipRequests: createdMembershipRequest._id,
         },
       },
+      { new: true, runValidators: true },
     );
 
     return createdMembershipRequest.toObject();