Record metrics for unsubscribe all success and failure

guardian · May 17, 2024 · 74b70bf · 74b70bf
1 parent cd5919f
commit 74b70bf
Show file tree

Hide file tree

Showing 2 changed files with 3 additions and 19 deletions.
diff --git a/src/server/models/Metrics.ts b/src/server/models/Metrics.ts
@@ -60,6 +60,7 @@ type ConditionalMetrics =
 	| 'SignOut'
 	| 'SignOutGlobal'
 	| 'Unsubscribe'
+	| 'UnsubscribeAll'
 	| 'Subscribe'
 	| 'UpdatePassword'
 	| 'RecaptchaMiddleware'

diff --git a/src/server/routes/subscriptions.ts b/src/server/routes/subscriptions.ts
@@ -102,32 +102,15 @@ router.post(
 				res.locals.requestId,
 			);
 
-			// TODO: metrics?
-			// trackMetric(`${subscriptionActionName(action)}::Success`);
-
-			// const html = renderer(`/${action}/success`, {
-			// 	requestState: mergeRequestState(res.locals, {
-			// 		pageData: buildPageData(emailType, subscriptionData.emailId),
-			// 	}),
-			// 	pageTitle: `${subscriptionActionName(action)} Confirmation`,
-			// });
+			trackMetric(`UnsubscribeAll::Success`);
 
 			return res.status(200).send();
 		} catch (error) {
 			logger.error(`${req.method} ${req.originalUrl}  Error`, error, {
 				request_id: res.locals.requestId,
 			});
 
-			// trackMetric(`${subscriptionActionName(action)}::Failure`);
-
-			// const html = renderer(`/${action}/error`, {
-			// 	requestState: mergeRequestState(res.locals, {
-			// 		pageData: {
-			// 			accountManagementUrl,
-			// 		},
-			// 	}),
-			// 	pageTitle: `${subscriptionActionName(action)} Error`,
-			// });
+			trackMetric(`UnsubscribeAll::Failure`);
 
 			return res.status(500).send();
 		}