Merge branch 'remove_unconfimed_email_user_accounts' into architecture-refactor

Merge architecture <- remove_unconfirmed_email_user_accounts

With this merge, I can deal with potential merge conflicts locally, and
avoid having them on the upstream repo.

Hope that is clear.
Massiles.

Author: MassiGy

package-lock.json

@@ -28,6 +28,7 @@
     "express": "^4.17.1",
     "jsdoc": "^3.6.10",
     "jsonwebtoken": "^8.5.1",
+    "node-schedule": "^2.1.0",
     "nodemailer": "^6.7.2",
     "pg": "^8.7.1",
     "sequelize": "^6.3.5",

package.json

@@ -24,22 +24,23 @@ module.exports.remove_unconfirmed_email_users = async () => {
             {
                 confirmed_email: false,
             },
-            attributes: ['id'],
+            attributes: ['id', 'createdAt'],
             raw: true,
         },
     )
-
+  
     if (found_users != undefined && found_users.length) {
 
-        // at this stage found_users will look like [{id:1}, {id:2}]
-        // we want to turn it into an ids array which will look like [1,2]
+        // at this stage found_users will look like [ { id: 1, createdAt: '2022-03-26 22:23:43.002 +00:00' } ]
+        // we want to turn it into an ids array which will look like [1], if the createdAt refers to a date before a month ago
         const ids = [];
         found_users.forEach((element) => {
-            ids.push(element.id)
+            if (Date.parse(element.createdAt) < (Date.now() - 30 * 24 * 60 * 60 * 1000)) {
+                ids.push(element.id);
+            }
         })
 
 
-
         //remove all the data that belongs to a user with an unconfirmed email address
         await Promise.allSettled([
             Events.findAll(