Database Cleanup System Overall Done

Within these final tweaks, we are now able not only to purge user
records if confirmed_email is set to false, but to also check if they
are inactiv during the last month.

I created a check that will make sure that we wont delete the user just
if the confirmed_email is set to false and also the createAt attribute
refer to a date before one month ago. All of that by adding a simple if
statement, the simpler the better ( @supernova350  )

Please refer to me if something is wrong.

Thanks.
Massiles.

Author: MassiGy

src/background_jobs/unconfirmed_emails.js

@@ -13,7 +13,6 @@
 
 
 module.exports.remove_unconfirmed_email_users = async () => {
-    console.log("job executed !")
 
 
     // find and retreive all the users with an unconfirmed email address
@@ -25,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(