feat: add expandable location rows to ChecksTable

## Changes
- Add LocationResult type and locationResults field to client Check type
- Add expandable rows to ChecksTable showing per-location Globalping results (location name, status, response time, status code)
- Pass globalpingEnabled and locationLabels props from Details page to ChecksTable
- Add translation keys for expanded row columns

## Benefits
Users can now click a check row to see how each Globalping location
performed for that specific monitoring cycle, providing at-a-glance
multi-location visibility.

client/src/Pages/Uptime/Details/Components/ChecksTable.tsx

@@ -1,5 +1,7 @@
 import { Table, Pagination, StatusLabel } from "@/Components/v2/design-elements";
 import Box from "@mui/material/Box";
+import Stack from "@mui/material/Stack";
+import Typography from "@mui/material/Typography";
 import type { Header } from "@/Components/v2/design-elements/";
 import type { Check } from "@/Types/Check";
 
@@ -8,6 +10,7 @@ import { useTranslation } from "react-i18next";
 import { formatDateWithTz } from "@/Utils/TimeUtils";
 import type { RootState } from "@/Types/state";
 import { useSelector } from "react-redux";
+import { useTheme } from "@mui/material/styles";
 
 const getHeaders = (t: Function, uiTimezone: string) => {
 	const headers: Header<Check>[] = [
@@ -50,6 +53,8 @@ export const ChecksTable = ({
 	setPage,
 	rowsPerPage,
 	setRowsPerPage,
+	globalpingEnabled = false,
+	locationLabels = {},
 }: {
 	checks: Check[];
 	count: number;
@@ -57,9 +62,12 @@ export const ChecksTable = ({
 	setPage: (page: number) => void;
 	rowsPerPage: number;
 	setRowsPerPage: (rowsPerPage: number) => void;
+	globalpingEnabled?: boolean;
+	locationLabels?: Record<string, string>;
 }) => {
 	const navigate = useNavigate();
 	const { t } = useTranslation();
+	const theme = useTheme();
 	const uiTimezone = useSelector((state: RootState) => state.ui.timezone);
 	const headers = getHeaders(t, uiTimezone);
 
@@ -78,6 +86,87 @@ export const ChecksTable = ({
 		setRowsPerPage(value);
 	};
 
+	const renderExpandedContent = (row: Check) => {
+		if (!row.locationResults || row.locationResults.length === 0) {
+			return null;
+		}
+
+		return (
+			<Stack gap={theme.spacing(2)}>
+				<Stack
+					direction="row"
+					sx={{
+						px: theme.spacing(4),
+						py: theme.spacing(2),
+						borderBottom: `1px solid ${theme.palette.divider}`,
+					}}
+				>
+					<Typography
+						variant="body2"
+						fontWeight={600}
+						sx={{ flex: 2 }}
+					>
+						{t("pages.checks.expandedRow.location")}
+					</Typography>
+					<Typography
+						variant="body2"
+						fontWeight={600}
+						sx={{ flex: 1 }}
+					>
+						{t("common.table.headers.status")}
+					</Typography>
+					<Typography
+						variant="body2"
+						fontWeight={600}
+						sx={{ flex: 1 }}
+					>
+						{t("pages.checks.expandedRow.responseTime")}
+					</Typography>
+					<Typography
+						variant="body2"
+						fontWeight={600}
+						sx={{ flex: 1 }}
+					>
+						{t("pages.checks.expandedRow.statusCode")}
+					</Typography>
+				</Stack>
+				{row.locationResults.map((lr) => (
+					<Stack
+						key={lr.location}
+						direction="row"
+						sx={{
+							px: theme.spacing(4),
+							py: theme.spacing(1),
+							alignItems: "center",
+						}}
+					>
+						<Typography
+							variant="body2"
+							sx={{ flex: 2 }}
+						>
+							{locationLabels[lr.location] ?? lr.location}
+						</Typography>
+						<Box sx={{ flex: 1 }}>
+							<StatusLabel status={lr.status ? "up" : "down"} />
+						</Box>
+						<Typography
+							variant="body2"
+							sx={{ flex: 1 }}
+						>
+							{Math.round(lr.responseTime)} ms
+						</Typography>
+						<Typography
+							variant="body2"
+							sx={{ flex: 1 }}
+						>
+							{lr.statusCode || "N/A"}
+						</Typography>
+					</Stack>
+				))}
+			</Stack>
+		);
+	};
+
 	return (
 		<Box>
 			<Table
@@ -86,6 +175,8 @@ export const ChecksTable = ({
 				onRowClick={(row) => {
 					navigate(`/checks/${row.id}`);
 				}}
+				expandableRows={globalpingEnabled}
+				renderExpandedContent={globalpingEnabled ? renderExpandedContent : undefined}
 			/>
 			<Pagination
 				component="div"

client/src/Pages/Uptime/Details/index.tsx

@@ -245,6 +245,8 @@ const UptimeDetailsPage = () => {
 				setPage={setPage}
 				rowsPerPage={rowsPerPage}
 				setRowsPerPage={setRowsPerPage}
+				globalpingEnabled={monitor?.globalpingEnabled ?? false}
+				locationLabels={locationLabels}
 			/>
 		</BasePage>
 	);

client/src/Types/Check.ts

@@ -117,6 +117,14 @@ export interface CheckTimings {
 	};
 }
 
+export interface LocationResult {
+	location: string;
+	status: boolean;
+	responseTime: number;
+	statusCode: number;
+	message: string;
+}
+
 export interface Check {
 	id: string;
 	metadata: CheckMetadata;
@@ -140,6 +148,7 @@ export interface Check {
 	seo?: number;
 	performance?: number;
 	audits?: CheckAudits;
+	locationResults?: LocationResult[];
 	__v: number;
 	createdAt: string;
 	updatedAt: string;

client/src/locales/en.json

@@ -368,6 +368,11 @@
 					"dateTime": "Date & time",
 					"statusCode": "Status code"
 				}
+			},
+			"expandedRow": {
+				"location": "Location",
+				"responseTime": "Response time",
+				"statusCode": "Status code"
 			}
 		},
 		"common": {

server/src/db/models/Check.ts

@@ -20,7 +20,6 @@ type CheckMetadataDocument = Omit<CheckMetadata, "monitorId" | "teamId"> & {
 	monitorId: Types.ObjectId;
 	teamId: Types.ObjectId;
 	type: MonitorType;
-	location?: string;
 };
 
 type CheckDocumentBase = Omit<Check, "id" | "metadata" | "expiry" | "ackAt" | "createdAt" | "updatedAt"> & {
@@ -210,9 +209,6 @@ const metadataSchema = new Schema<CheckMetadataDocument>(
 			required: true,
 			index: true,
 		},
-		location: {
-			type: String,
-		},
 	},
 	{ _id: false }
 );
@@ -319,7 +315,6 @@ CheckSchema.index({ createdAt: 1 });
 CheckSchema.index({ "metadata.teamId": 1, createdAt: -1 });
 CheckSchema.index({ "metadata.monitorId": 1, "metadata.type": 1, createdAt: -1 });
 CheckSchema.index({ "metadata.teamId": 1, status: 1, createdAt: -1 });
-CheckSchema.index({ "metadata.monitorId": 1, "metadata.location": 1, createdAt: -1 });
 
 const CheckModel = model<CheckDocument>("Check", CheckSchema);
 

server/src/repositories/checks/IChecksRepository.ts

@@ -2,6 +2,7 @@ import type {
 	Check,
 	ChecksQueryResult,
 	ChecksSummary,
+	LocationResult,
 	MonitorType,
 	PageSpeedChecksResult,
 	HardwareChecksResult,
@@ -37,6 +38,7 @@ export interface IChecksRepository {
 	findSummaryByTeamId(teamId: string, dateRange: string): Promise<ChecksSummary>;
 	findLocationChecksByMonitorId(monitorId: string, startDate: Date, endDate: Date, dateString: string): Promise<Record<string, UptimeChecksResult>>;
 	// update
+	updateLocationResults(checkId: string, locationResults: LocationResult[]): Promise<void>;
 	//delete
 	deleteByMonitorId(monitorId: string): Promise<number>;
 	deleteByTeamId(teamId: string): Promise<number>;

server/src/repositories/checks/MongoChecksRepistory.ts

@@ -11,6 +11,7 @@ import type {
 	CheckMetadata,
 	CheckNetworkInterfaceInfo,
 	GotTimings,
+	LocationResult,
 	MonitorType,
 } from "@/types/index.js";
 import { CheckModel, MonitorModel, type CheckDocument } from "@/db/models/index.js";
@@ -169,7 +170,6 @@ class MongoChecksRepository implements IChecksRepository {
 			monitorId: toStringId(metadata.monitorId),
 			teamId: toStringId(metadata.teamId),
 			type: metadata.type,
-			...(metadata.location ? { location: metadata.location } : {}),
 		});
 
 		return {
@@ -195,6 +195,7 @@ class MongoChecksRepository implements IChecksRepository {
 			seo: doc.seo,
 			performance: doc.performance,
 			audits: mapAudits(doc.audits),
+			locationResults: (doc as any).locationResults,
 			__v: doc.__v ?? 0,
 			createdAt: toDateString(doc.createdAt),
 			updatedAt: toDateString(doc.updatedAt),
@@ -220,7 +221,6 @@ class MongoChecksRepository implements IChecksRepository {
 				monitorId: new mongoose.Types.ObjectId(metadata.monitorId),
 				teamId: new mongoose.Types.ObjectId(metadata.teamId),
 				type: metadata.type,
-				...(metadata.location ? { location: metadata.location } : {}),
 			},
 			...rest,
 		} as unknown as CheckDocument;
@@ -406,6 +406,10 @@ class MongoChecksRepository implements IChecksRepository {
 		};
 	};
 
+	updateLocationResults = async (checkId: string, locationResults: LocationResult[]): Promise<void> => {
+		await CheckModel.updateOne({ _id: new mongoose.Types.ObjectId(checkId) }, { $set: { locationResults } });
+	};
+
 	deleteByMonitorId = async (monitorId: string): Promise<number> => {
 		const result = await CheckModel.deleteMany({ "metadata.monitorId": new mongoose.Types.ObjectId(monitorId) });
 		return result.deletedCount;
@@ -424,24 +428,25 @@ class MongoChecksRepository implements IChecksRepository {
 
 	findLocationChecksByMonitorId = async (monitorId: string, startDate: Date, endDate: Date, dateString: string): Promise<Record<string, any>> => {
 		const monitorObjectId = new mongoose.Types.ObjectId(monitorId);
-		const matchStage = {
-			"metadata.monitorId": monitorObjectId,
-			"metadata.location": { $exists: true, $ne: null },
-			createdAt: { $gte: startDate, $lte: endDate },
-		};
 
 		const results = await CheckModel.aggregate([
-			{ $match: matchStage },
-			{ $sort: { createdAt: 1 } },
+			{
+				$match: {
+					"metadata.monitorId": monitorObjectId,
+					createdAt: { $gte: startDate, $lte: endDate },
+					locationResults: { $type: "array", $ne: [] },
+				},
+			},
+			{ $unwind: "$locationResults" },
 			{
 				$group: {
 					_id: {
-						location: "$metadata.location",
+						location: "$locationResults.location",
 						bucket: { $dateToString: { format: dateString, date: "$createdAt" } },
 					},
-					avgResponseTime: { $avg: "$responseTime" },
+					avgResponseTime: { $avg: "$locationResults.responseTime" },
 					totalChecks: { $sum: 1 },
-					upChecks: { $sum: { $cond: [{ $eq: ["$status", true] }, 1, 0] } },
+					upChecks: { $sum: { $cond: [{ $eq: ["$locationResults.status", true] }, 1, 0] } },
 				},
 			},
 			{ $sort: { "_id.bucket": 1 } },
@@ -485,7 +490,6 @@ class MongoChecksRepository implements IChecksRepository {
 		const matchStage: Record<string, any> = {
 			"metadata.monitorId": monitorObjectId,
 			createdAt: { $gte: startDate, $lte: endDate },
-			"metadata.location": { $exists: false },
 		};
 		const [result] = await CheckModel.aggregate([
 			{ $match: matchStage },

server/src/service/business/checkService.ts

@@ -37,7 +37,7 @@ class CheckService {
 	};
 
 	buildCheck = (statusResponse: MonitorStatusResponse<PageSpeedStatusPayload | HardwareStatusPayload | undefined>) => {
-		const { monitorId, teamId, type, status, responseTime, code, message, payload, timings, location } = statusResponse;
+		const { monitorId, teamId, type, status, responseTime, code, message, payload, timings } = statusResponse;
 
 		const check: Partial<Check> = {
 			id: new Types.ObjectId().toString(),
@@ -46,7 +46,6 @@ class CheckService {
 				monitorId,
 				teamId,
 				type,
-				...(location ? { location } : {}),
 			},
 			status,
 			statusCode: code,

server/src/service/infrastructure/SuperSimpleQueue/SuperSimpleQueueHelper.ts

@@ -127,33 +127,6 @@ class SuperSimpleQueueHelper {
 					throw new Error("No network response");
 				}
 
-				// Step 2b. Run Globalping checks (fire-and-forget, doesn't affect local check pipeline)
-				if (monitor.globalpingEnabled && (monitor.type === "http" || monitor.type === "ping")) {
-					this.globalpingService
-						.runChecks(monitor)
-						.then((locationStatuses) => {
-							for (const locStatus of locationStatuses) {
-								const locCheck = this.checkService.buildCheck(locStatus);
-								if (locCheck) {
-									this.buffer.addToBuffer(locCheck);
-								} else {
-									this.logger.warn({
-										message: `Failed to build Globalping check for monitor ${monitorId}, location: ${locStatus.location}`,
-										service: SERVICE_NAME,
-										method: "getMonitorJob",
-									});
-								}
-							}
-						})
-						.catch((err: any) => {
-							this.logger.warn({
-								message: `Globalping checks failed for monitor ${monitorId}: ${err.message}`,
-								service: SERVICE_NAME,
-								method: "getMonitorJob",
-							});
-						});
-				}
-
 				// Step 3. Build check
 				const check = await this.checkService.buildCheck(status);
 				if (!check) {
@@ -167,6 +140,38 @@ class SuperSimpleQueueHelper {
 				}
 				// Step 4. Add check to buffer
 				this.buffer.addToBuffer(check);
+
+				// Step 4b. Run Globalping checks and backfill into the same check document.
+				// Fire-and-forget: Globalping results are supplementary and must not block the
+				// local check pipeline. The dual-write strategy below is safe against mid-flush
+				// races because flushBuffer() replaces the buffer array reference atomically
+				// (this.buffer = []) — if findInBuffer returns a hit, the check is still in the
+				// current buffer and our mutation will be included in the upcoming flush; if it
+				// returns undefined the check has already been flushed, so we fall back to a
+				// direct DB update.
+				if (monitor.globalpingEnabled && (monitor.type === "http" || monitor.type === "ping") && check.id) {
+					const checkId = check.id;
+					this.globalpingService
+						.runChecks(monitor)
+						.then(async (locationResults) => {
+							if (locationResults.length > 0) {
+								const buffered = this.buffer.findInBuffer(checkId);
+								if (buffered) {
+									buffered.locationResults = locationResults;
+								} else {
+									await this.checksRepository.updateLocationResults(checkId, locationResults);
+								}
+							}
+						})
+						.catch((err: any) => {
+							this.logger.warn({
+								message: `Globalping checks failed for monitor ${monitorId}: ${err.message}`,
+								service: SERVICE_NAME,
+								method: "getMonitorJob",
+							});
+						});
+				}
+
 				// Step 5. Update monitor status
 				const statusChangeResult = await this.statusService.updateMonitorStatus(status, check);
 

server/src/service/infrastructure/bufferService.ts

@@ -5,6 +5,7 @@ const SERVICE_NAME = "BufferService";
 export interface IBufferService {
 	addToBuffer(check: Check): void;
 	removeCheckFromBuffer(check: Check): boolean;
+	findInBuffer(checkId: string): Check | undefined;
 	scheduleNextFlush(): void;
 	flushBuffer(): Promise<void>;
 }
@@ -49,6 +50,10 @@ class BufferService implements IBufferService {
 		}
 	}
 
+	findInBuffer(checkId: string): Check | undefined {
+		return this.buffer.find((c: Check) => c.id?.toString() === checkId);
+	}
+
 	removeCheckFromBuffer(checkToRemove: Check) {
 		try {
 			if (!checkToRemove) {

server/src/service/infrastructure/globalpingService.ts

@@ -1,7 +1,7 @@
 import { Globalping } from "globalping";
 import type { ISettingsService } from "@/service/system/settingsService.js";
 import type { Monitor } from "@/types/monitor.js";
-import type { MonitorStatusResponse } from "@/types/network.js";
+import type { GlobalpingCheckResult } from "@/types/network.js";
 
 const SERVICE_NAME = "GlobalpingService";
 
@@ -130,7 +130,7 @@ class GlobalpingService {
 		return { client: this.client, locationIds };
 	}
 
-	async runChecks(monitor: Monitor): Promise<MonitorStatusResponse[]> {
+	async runChecks(monitor: Monitor): Promise<GlobalpingCheckResult[]> {
 		if (monitor.type !== "http" && monitor.type !== "ping") {
 			return [];
 		}
@@ -148,7 +148,7 @@ class GlobalpingService {
 		}
 	}
 
-	private async executeChecks(monitor: Monitor): Promise<MonitorStatusResponse[]> {
+	private async executeChecks(monitor: Monitor): Promise<GlobalpingCheckResult[]> {
 		const { client, locationIds } = await this.getClientAndLocations();
 		if (!client || locationIds.length === 0) {
 			return [];
@@ -224,7 +224,7 @@ class GlobalpingService {
 			}
 
 			const measurement = awaitResult.data as any;
-			const results: MonitorStatusResponse[] = [];
+			const results: GlobalpingCheckResult[] = [];
 
 			for (const item of measurement.results || []) {
 				const probe = item.probe;
@@ -233,27 +233,21 @@ class GlobalpingService {
 
 				if (measurementType === "http") {
 					results.push({
-						monitorId: monitor.id,
-						teamId: monitor.teamId,
-						type: monitor.type,
+						location: locationId,
 						status: result.statusCode >= 200 && result.statusCode < 400,
-						code: result.statusCode || 0,
+						statusCode: result.statusCode || 0,
 						message: result.statusCodeName || result.rawOutput || "",
 						responseTime: result.timings?.total ?? 0,
-						location: locationId,
 					});
 				} else {
 					// Ping: loss < 100% is considered up (consistent with local ping checks)
 					const stats = result.stats;
 					results.push({
-						monitorId: monitor.id,
-						teamId: monitor.teamId,
-						type: monitor.type,
+						location: locationId,
 						status: stats?.loss !== undefined ? stats.loss < 100 : false,
-						code: stats?.loss === 0 ? 200 : 5000,
+						statusCode: stats?.loss === 0 ? 200 : 5000,
 						message: stats?.loss === 0 ? "Ping successful" : `Packet loss: ${stats?.loss}%`,
 						responseTime: stats?.avg ?? 0,
-						location: locationId,
 					});
 				}
 			}

server/src/types/check.ts

@@ -6,7 +6,6 @@ export interface CheckMetadata {
 	monitorId: string;
 	teamId: string;
 	type: MonitorType;
-	location?: string;
 }
 
 export interface LocationResult {

server/src/types/network.ts

@@ -34,7 +34,6 @@ export interface MonitorStatusResponse<T = any> {
 	matchMethod?: MonitorMatchMethod;
 	expectedValue?: string;
 	extracted?: unknown;
-	location?: string;
 }
 
 export interface GlobalpingCheckResult {