Revert "Merge branch 'develop' of https://github.com/PalisadoesFounda…

…tion/talawa-admin into unseen-msgs"

This reverts commit c20ae65, reversing
changes made to da51531.

.env.example

@@ -24,8 +24,8 @@ REACT_APP_USE_RECAPTCHA=
 REACT_APP_RECAPTCHA_SITE_KEY=
 
 # has to be inserted in the env file to use plugins and other websocket based features.
-REACT_APP_BACKEND_WEBSOCKET_URL=ws://localhost:4000/graphql/
+REACT_APP_BACKEND_WEBSOCKET_URL=ws://localhost:4000/graphql
 
 # If you want to logs Compiletime and Runtime error , warning and info write YES or if u want to  
 # keep the console clean leave it blank
-ALLOW_LOGS=
+ALLOW_LOGS=

.eslintignore

@@ -1,8 +1,2 @@
 # Contains the PDF file of the Tag as JSON string, thus does not need to be linted
-src/components/CheckIn/tagTemplate.ts
-package.json
-package-lock.json
-tsconfig.json
-docker-compose.yml
-Dockerfile
-nginx.conf
+src/components/CheckIn/tagTemplate.ts

.eslintrc.json

@@ -5,6 +5,7 @@
     "es6": true
   },
 
+  // Specify the rulesets from other ESLint Plugins tobe used
   "extends": [
     "plugin:react/recommended",
     "eslint:recommended",
@@ -27,6 +28,7 @@
     "sourceType": "module"
   },
 
+  // Specify the ESLint plugins tobe used
   "plugins": [
     "react",
     "@typescript-eslint",
@@ -53,7 +55,6 @@
     "import/no-duplicates": "error",
     "tsdoc/syntax": "error",
     "@typescript-eslint/ban-ts-comment": "error",
-    "@typescript-eslint/no-unused-vars": "error",
     "@typescript-eslint/no-explicit-any": "error",
     "@typescript-eslint/no-inferrable-types": "error",
     "@typescript-eslint/no-non-null-asserted-optional-chain": "error",
@@ -76,13 +77,13 @@
     "camelcase": "off",
     "@typescript-eslint/naming-convention": [
       "error",
-
+      // Interfaces must begin with Interface or TestInterface followed by a PascalCase name
       {
         "selector": "interface",
         "format": ["PascalCase"],
         "prefix": ["Interface", "TestInterface"]
       },
-
+      // Type Aliases must be in PascalCase
       {
         "selector": ["typeAlias", "typeLike", "enum"],
         "format": ["PascalCase"]
@@ -119,17 +120,20 @@
         "format": null
       }
     ],
-
+    // Ensures that components are always written in PascalCase
     "react/jsx-pascal-case": [
       "error",
       { "allowAllCaps": false, "allowNamespace": false }
     ],
 
+    // Enforces whitespace around equal sign operators
     "react/jsx-equals-spacing": ["warn", "never"],
     "react/no-this-in-sfc": "error",
 
+    // All tests must need not have an assertion
     "jest/expect-expect": 0,
 
+    // Enforce Strictly functional components
     "react/no-unstable-nested-components": ["error", { "allowAsProps": true }],
     "react/function-component-definition": [
       0,
@@ -138,10 +142,10 @@
     "prettier/prettier": "error"
   },
 
+  // Let ESLint use the react version in the package.json
   "settings": {
     "react": {
       "version": "detect"
     }
-  },
-  "ignorePatterns": ["**/*.css", "**/*.scss", "**/*.less", "**/*.json"]
+  }
 }

.github/dependabot.yaml

@@ -7,12 +7,12 @@ updates:
     directory: "/"
     # Schedule automated updates to run weekly
     schedule:
-      interval: "monthly"
+      interval: "weekly"
     # Labels to apply to Dependabot PRs
     labels:
       - "dependencies"
     # Specify the target branch for PRs
     target-branch: "develop"
     # Customize commit message prefix
     commit-message:
-      prefix: "chore(deps):"
+      prefix: "chore(deps):"

.github/pull_request_template.md

@@ -1,4 +1,3 @@
-
 <!--
 This section can be deleted after reading.
 
@@ -10,7 +9,6 @@ We employ the following branching strategy to simplify the development process a
 NOTE!!!
 
 ONLY SUBMIT PRS AGAINST OUR `DEVELOP` BRANCH. THE DEFAULT IS `MAIN`, SO YOU WILL HAVE TO MODIFY THIS BEFORE SUBMITTING YOUR PR FOR REVIEW. PRS MADE AGAINST `MAIN` WILL BE CLOSED.
-
 -->
 
 <!--
@@ -25,6 +23,10 @@ Thanks for submitting a pull request! Please provide enough information so that
 
 Fixes #<!--Add related issue number here.-->
 
+**Did you add tests for your changes?**
+
+<!--Yes or No. Note: Add unit tests or automation tests for your code.-->
+
 **Snapshots/Videos:**
 
 <!--Add snapshots or videos wherever possible.-->
@@ -42,23 +44,10 @@ Fixes #<!--Add related issue number here.-->
 
 <!-- If this PR introduces a breaking change, please describe the impact and a migration path for existing applications. -->
 
-## Checklist
-
-### CodeRabbit AI Review
-- [ ] I have reviewed and addressed all critical issues flagged by CodeRabbit AI
-- [ ] I have implemented or provided justification for each non-critical suggestion
-- [ ] I have documented my reasoning in the PR comments where CodeRabbit AI suggestions were not implemented
-
-### Test Coverage
-- [ ] I have written tests for all new changes/features
-- [ ] I have verified that test coverage meets or exceeds 95%
-- [ ] I have run the test suite locally and all tests pass
-
-
 **Other information**
 
 <!--Add extra information about this PR here-->
 
 **Have you read the [contributing guide](https://github.com/PalisadoesFoundation/talawa-admin/blob/master/CONTRIBUTING.md)?**
 
-<!--Yes or No-->
+<!--Yes or No-->

.github/workflows/check-tsdoc.js

@@ -23,7 +23,6 @@ async function findTsxFiles(dir) {
       } else if (
         filePath.endsWith('.tsx') &&
         !filePath.endsWith('.test.tsx') &&
-        !filePath.endsWith('.spec.tsx') &&
         !filesToSkip.includes(path.relative(dir, filePath))
       ) {
         results.push(filePath);

.github/workflows/codeql-codescan.yml

@@ -32,13 +32,13 @@ jobs:
          uses: actions/checkout@v4
 
        - name: Initialize CodeQL
-         uses: github/codeql-action/init@v3
+         uses: github/codeql-action/init@v2
          with:
           languages: ${{ matrix.language }}
           debug: true
 
        - name: Autobuild
-         uses: github/codeql-action/autobuild@v3
+         uses: github/codeql-action/autobuild@v2
 
        - name: Perform CodeQL Analysis
-         uses: github/codeql-action/analyze@v3
+         uses: github/codeql-action/analyze@v2

.github/workflows/issue.yml

@@ -18,43 +18,12 @@ jobs:
     name: Adding Issue Label
     runs-on: ubuntu-latest
     steps:
-      - uses: actions/checkout@v4
-        with:
-          sparse-checkout: |
-            .github/workflows/auto-label.json5
-          sparse-checkout-cone-mode: false
-      - uses: Renato66/auto-label@v3
+      - uses: Renato66/[email protected]
         with:
           repo-token: ${{ secrets.GITHUB_TOKEN }}
-      - uses: actions/github-script@v7
-        env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
-        with:
-          script: |
-            const { owner, repo } = context.repo;
-            const issue_number = context.issue.number;
-            const apiParams = {
-              owner,
-              repo,
-              issue_number
-            };
-            const labels = await github.rest.issues.listLabelsOnIssue(apiParams);
-            if(labels.data.reduce((a, c)=>a||["dependencies"].includes(c.name), false))
-              await github.rest.issues.addLabels({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                issue_number: context.issue.number,
-                labels: ["good first issue", "security"]
-              });
-            else if(labels.data.reduce((a, c)=>a||["security", "ui/ux"].includes(c.name), false))
-              await github.rest.issues.addLabels({
-                owner: context.repo.owner,
-                repo: context.repo.repo,
-                issue_number: context.issue.number,
-                labels: ["good first issue"]
-              });
-
-
+          ignore-comments: true
+          default-labels: '["unapproved"]'
+
   Issue-Greeting:
     name: Greeting Message to User
     runs-on: ubuntu-latest

.github/workflows/pull-request-target.yml

@@ -19,26 +19,23 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Add the PR Review Policy
-        uses: thollander/actions-comment-pull-request@v3
+        uses: thollander/actions-comment-pull-request@v2
         with:
-          comment-tag: pr_review_policy
+          comment_tag: pr_review_policy
           message: |
             ## Our Pull Request Approval Process
-            
-            Thanks for contributing! 
-            
+
+            We have these basic policies to make the approval process smoother for our volunteer team.
+
             ### Testing Your Code
-            Remember, your PRs won't be reviewed until these criteria are met:
-
-            1. We don't merge PRs with poor code quality. 
-                1. Follow coding best practices such that CodeRabbit.ai approves your PR.
-            1. We don't merge PRs with failed tests. 
-                1. When tests fail, click on the `Details` link to learn more.
-                1. Write sufficient tests for your changes (CodeCov Patch Test). Your testing level must be better than the target threshold of the repository
-                1. Tests may fail if you edit sensitive files. Ask to add the `ignore-sensitive-files-pr` label if the edits are necessary.
-            1. We cannot merge PRs with conflicting files. These must be fixed.
-                
-            Our policies make our code better.
+
+            Please make sure your code passes all tests. Our test code coverage system will fail if these conditions occur:
+
+            1. The overall code coverage drops below the target threshold of the repository
+            2. Any file in the pull request has code coverage levels below the repository threshold
+            3. Merge conflicts 
+
+            The process helps maintain the overall reliability of the code base and is a prerequisite for getting your PR approved. Assigned reviewers regularly review the PR queue and tend to focus on PRs that are passing.
 
             ### Reviewers