Merge pull request #2550 from bluewave-labs/hotfix/status-page-with-no-monitors

hotfix: status page

client/src/Pages/StatusPage/Create/index.jsx

@@ -246,7 +246,7 @@ const CreateStatusPage = () => {
 					color="accent"
 					onClick={handleSubmit}
 				>
-					{t("settingsSave")}
+					{t("statusPageCreate.buttonSave")}
 				</Button>
 			</Stack>
 		</Stack>

client/src/Pages/StatusPage/Status/index.jsx

@@ -62,7 +62,7 @@ const PublicStatus = () => {
 		return <SkeletonLayout />;
 	}
 
-	if (monitors.length === 0) {
+	if (monitors?.length === 0) {
 		return (
 			<GenericFallback>
 				<Typography

client/src/locales/en.json

@@ -835,5 +835,8 @@
 			"selectEnabled": "Enabled",
 			"title": "Monitor IP/URL on Status Page"
 		}
+	},
+	"statusPageCreate": {
+		"buttonSave": "Save"
 	}
 }

server/db/models/Monitor.js

@@ -147,6 +147,7 @@ MonitorSchema.pre("findOneAndDelete", async function (next) {
 MonitorSchema.pre("deleteMany", async function (next) {
 	const filter = this.getFilter();
 	const monitors = await this.model.find(filter).select(["_id", "type"]).lean();
+
 	for (const monitor of monitors) {
 		if (monitor.type === "pagespeed") {
 			await PageSpeedCheck.deleteMany({ monitorId: monitor._id });
@@ -155,6 +156,10 @@ MonitorSchema.pre("deleteMany", async function (next) {
 		} else {
 			await Check.deleteMany({ monitorId: monitor._id });
 		}
+		await StatusPage.updateMany(
+			{ monitors: monitor._id },
+			{ $pull: { monitors: monitor._id } }
+		);
 		await MonitorStats.deleteMany({ monitorId: monitor._id.toString() });
 	}
 	next();

server/db/mongo/modules/statusPageModule.js

@@ -90,6 +90,45 @@ const getStatusPage = async (url) => {
 	const stringService = ServiceRegistry.get(StringService.SERVICE_NAME);
 
 	try {
+		const preliminaryStatusPage = await StatusPage.findOne({ url });
+		if (!preliminaryStatusPage) {
+			const error = new Error(stringService.statusPageNotFound);
+			error.status = 404;
+			throw error;
+		}
+
+		if (!preliminaryStatusPage.monitors || preliminaryStatusPage.monitors.length === 0) {
+			const {
+				_id,
+				color,
+				companyName,
+				isPublished,
+				logo,
+				originalMonitors,
+				showCharts,
+				showUptimePercentage,
+				timezone,
+				showAdminLoginLink,
+				url,
+			} = preliminaryStatusPage;
+			return {
+				statusPage: {
+					_id,
+					color,
+					companyName,
+					isPublished,
+					logo,
+					originalMonitors,
+					showCharts,
+					showUptimePercentage,
+					timezone,
+					showAdminLoginLink,
+					url,
+				},
+				monitors: [],
+			};
+		}
+
 		const statusPageQuery = await StatusPage.aggregate([
 			{ $match: { url: url } },
 			{