fix(alerting): use alerting-admin as navId for settings (#108116)

This commit is contained in:
Konrad Lalik 2025-07-15 16:34:40 +02:00 committed by GitHub
parent 8cb6993fe6
commit 930c5a7024
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
2 changed files with 3 additions and 3 deletions

View File

@ -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

View File

@ -34,7 +34,7 @@ export function useSettingsPageNav() {
};
return {
navId: 'alerting',
navId: 'alerting-admin',
pageNav,
};
}