From 930c5a7024eef74e74b553efaaa846a2d83b9e39 Mon Sep 17 00:00:00 2001
From: Konrad Lalik <konradlalik@gmail.com>
Date: Tue, 15 Jul 2025 16:34:40 +0200
Subject: [PATCH] fix(alerting): use alerting-admin as navId for settings
 (#108116)

---
 .../app/features/alerting/unified/settings/navigation.test.ts | 4 ++--
 public/app/features/alerting/unified/settings/navigation.ts   | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)

diff --git a/public/app/features/alerting/unified/settings/navigation.test.ts b/public/app/features/alerting/unified/settings/navigation.test.ts
index c4b4c4678ef..380d747a8a6 100644
--- a/public/app/features/alerting/unified/settings/navigation.test.ts
+++ b/public/app/features/alerting/unified/settings/navigation.test.ts
@@ -28,7 +28,7 @@ describe('useSettingsPageNav', () => {
 
     const { result } = renderHook(() => useSettingsPageNav(), { wrapper });
 
-    expect(result.current.navId).toBe('alerting');
+    expect(result.current.navId).toBe('alerting-admin');
 
     // Check the structure
     // eslint-disable-next-line testing-library/no-node-access
@@ -75,7 +75,7 @@ describe('useSettingsPageNav', () => {
 
     const { result } = renderHook(() => useSettingsPageNav(), { wrapper });
 
-    expect(result.current.navId).toBe('alerting');
+    expect(result.current.navId).toBe('alerting-admin');
 
     // Should have 3 children: alertmanager + 2 extensions
     // eslint-disable-next-line testing-library/no-node-access
diff --git a/public/app/features/alerting/unified/settings/navigation.ts b/public/app/features/alerting/unified/settings/navigation.ts
index 85513269e1f..83c4afb69f0 100644
--- a/public/app/features/alerting/unified/settings/navigation.ts
+++ b/public/app/features/alerting/unified/settings/navigation.ts
@@ -34,7 +34,7 @@ export function useSettingsPageNav() {
   };
 
   return {
-    navId: 'alerting',
+    navId: 'alerting-admin',
     pageNav,
   };
 }