Merge pull request #2205 from bluewave-labs/hotfix/pagespeed-monitors-config

hotfix: pagespeed monitors config

client/src/Features/PageSpeedMonitor/pageSpeedMonitorSlice.js

@@ -101,7 +101,7 @@ export const updatePageSpeed = createAsyncThunk(
 				name: monitor.name,
 				description: monitor.description,
 				interval: monitor.interval,
-				// notifications: monitor.notifications,
+				notifications: monitor.notifications,
 			};
 			const res = await networkService.updateMonitor({
 				monitorId: monitor._id,

client/src/Pages/PageSpeed/Monitors/index.jsx

@@ -62,7 +62,7 @@ const PageSpeed = () => {
 			<Breadcrumbs list={BREADCRUMBS} />
 			<CreateMonitorHeader
 				isAdmin={isAdmin}
-				shouldRender={!isLoading}
+				isLoading={isLoading}
 				path="/pagespeed/create"
 			/>
 			<MonitorCountHeader

server/controllers/monitorController.js

@@ -538,18 +538,16 @@ class MonitorController {
 			const monitorBeforeEdit = await this.db.getMonitorById(monitorId);
 
 			// Get notifications from the request body
-			const notifications = req.body.notifications;
-
+			const notifications = req.body.notifications ?? [];
 			const editedMonitor = await this.db.editMonitor(monitorId, req.body);
 
 			await this.db.deleteNotificationsByMonitorId(editedMonitor._id);
 
 			await Promise.all(
-				notifications &&
-					notifications.map(async (notification) => {
-						notification.monitorId = editedMonitor._id;
-						await this.db.createNotification(notification);
-					})
+				notifications.map(async (notification) => {
+					notification.monitorId = editedMonitor._id;
+					await this.db.createNotification(notification);
+				})
 			);
 
 			// Delete the old job(editedMonitor has the same ID as the old monitor)