Merge pull request #5717 from Countly/SER-1747-alerts-table-default-order-should-be-by-creation-time-newest-at-the-top

[SER-1747] alerts table default order should be by creation time newest at the top

Author: ar2rsawseen

CHANGELOG.md

@@ -1,3 +1,7 @@
+## Version 25.x.x
+Features:
+- [alerts] alerts table default order should be by creation time newest at the top
+
 ## Version 24.12.x
 
 Fixes:

bin/upgrade/DEV/add_creation_date_for_existing_alerts.js

@@ -0,0 +1,25 @@
+// Script that adds creation date for existing alerts.
+
+const pluginManager = require('../../../plugins/pluginManager.js');
+
+pluginManager.dbConnection().then(async(countlyDb) => {
+    try {
+        await countlyDb.collection('alerts').updateMany(
+            { createdAt: { $exists: false } },
+            [
+                {
+                    $set: {
+                        createdAt: { $toDouble: { $toDate: "$_id" } }
+                    }
+                }
+            ]
+        );
+        console.log("Finished adding creation date for existing alerts.");
+    }
+    catch (error) {
+        console.log(`Error adding creation date for existing alerts: ${error}`);
+    }
+    finally {
+        countlyDb.close();
+    }
+});

plugins/alerts/api/api.js

@@ -229,6 +229,9 @@ const PERIOD_TO_TEXT_EXPRESSION_MAPPER = {
                             }
                         });
                 }
+                if (!alertConfig._id) {
+                    alertConfig.createdAt = new Date().getTime();
+                }
                 alertConfig.createdBy = params.member._id;
                 return common.db.collection("alerts").insert(
                     alertConfig,

plugins/alerts/frontend/public/javascripts/countly.models.js

@@ -503,6 +503,7 @@
                                 enabled: alertsList[i].enabled || false,
                                 createdByUser:
                                     alertsList[i].createdByUser || "",
+                                createdAt: alertsList[i].createdAt || "",
                                 _canUpdate: countlyAuth.validateUpdate(
                                     FEATURE_NAME,
                                     countlyGlobal.member,

plugins/alerts/frontend/public/templates/vue-main.html

@@ -44,7 +44,9 @@
                 class="cly-vue-alerts-table"
                 :tracked-fields="localTableTrackedFields"
                 :rows="tableRows"
-                :resizable="false">
+                :resizable="false"
+                :default-sort="{prop: 'createdAt', order: 'descending'}"
+            >
                 <template v-slot:header-left="scope">
                     <div class="alerts-table-app-selector">
                         <cly-select-x