dohertj2/scadalink-design
1
0
Fork 0
Code Issues Pull Requests 1 Actions Packages Projects Releases Wiki Activity

refactor(ui): rename AuditLog viewer to ConfigurationAuditLog under /audit/configuration (#23 M7)

Browse Source 
The pre-M1 IAuditService config-change viewer moves out of the Monitoring
nav group to make room for the new Audit nav group (issue #23 M7). The
old route /monitoring/audit-log returns 404 (no redirect, per plan) — the
viewer is now reachable at /audit/configuration and labelled
"Configuration Audit Log" to disambiguate from the new Audit Log page
(arriving in #23 M7-T9). Inbound references in NavMenu, Dashboard, and
the Playwright nav tests are updated to the new route and label.
This commit is contained in:
Joseph Doherty
2026-05-20 19:46:09 -04:00
parent 2d13886286
commit a9f45b0861
5 changed files with 9 additions and 10 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
src/ScadaLink.CentralUI/Components/Layout/NavMenu.razor
View File

@@ -108,11 +108,11 @@
</Authorized>
</AuthorizeView>
@* Audit Log — Admin only *@
@* Configuration Audit Log — Admin only *@
<AuthorizeView Policy="@AuthorizationPolicies.RequireAdmin">
<Authorized Context="auditContext">
<li class="nav-item">
<NavLink class="nav-link" href="/monitoring/audit-log">Audit Log</NavLink>
<NavLink class="nav-link" href="/audit/configuration">Configuration Audit Log</NavLink>
</li>
</Authorized>
</AuthorizeView>

4
src/ScadaLink.CentralUI/Components/Pages/Monitoring/AuditLog.razor → src/ScadaLink.CentralUI/Components/Pages/Audit/ConfigurationAuditLog.razor
View File

@@ -1,4 +1,4 @@
@page "/monitoring/audit-log"
@page "/audit/configuration"
@using ScadaLink.Security
@using ScadaLink.CentralUI.Components
@using ScadaLink.Commons.Entities.Audit
@@ -8,7 +8,7 @@
@inject IJSRuntime JS
<div class="container-fluid mt-3">
<h4 class="mb-3">Audit Log</h4>
<h4 class="mb-3">Configuration Audit Log</h4>
<ToastNotification @ref="_toast" />

4
src/ScadaLink.CentralUI/Components/Pages/Dashboard.razor
View File

@@ -70,10 +70,10 @@
</a>
</div>
<div class="col-lg-4 col-md-6 col-12">
<a class="card h-100 text-decoration-none text-reset" href="/monitoring/audit-log">
<a class="card h-100 text-decoration-none text-reset" href="/audit/configuration">
<div class="card-body">
<div class="d-flex justify-content-between align-items-start">
<h6 class="mb-1">Recent Audit Log</h6>
<h6 class="mb-1">Configuration Audit Log</h6>
<span class="text-muted">&rarr;</span>
</div>
<p class="text-muted small mb-0">Browse changes to configuration and deployments.</p>

1
tests/ScadaLink.CentralUI.PlaywrightTests/NavigationTests.cs
View File

@@ -68,7 +68,6 @@ public class NavigationTests
[InlineData("Health Dashboard", "/monitoring/health")]
[InlineData("Event Logs", "/monitoring/event-logs")]
[InlineData("Parked Messages", "/monitoring/parked-messages")]
[InlineData("Audit Log", "/monitoring/audit-log")]
public async Task MonitoringNavLinks_NavigateCorrectly(string linkText, string expectedPath)
{
var page = await _fixture.NewAuthenticatedPageAsync();

6
tests/ScadaLink.CentralUI.PlaywrightTests/RoleNavigationTests.cs
View File

@@ -158,16 +158,16 @@ public class RoleNavigationTests
}
[Fact]
public async Task DeploymentUser_SeesMonitoringButNotAuditLog()
public async Task DeploymentUser_SeesMonitoringButNotConfigurationAuditLog()
{
var page = await _fixture.NewAuthenticatedPageAsync("deployer", "password");
// Event Logs and Parked Messages are Deployment-role gated, so a
// Deployment user sees them; Audit Log is Admin-only.
// Deployment user sees them; Configuration Audit Log is Admin-only.
await AssertNavLinkVisible(page, "Health Dashboard");
await AssertNavLinkVisible(page, "Event Logs");
await AssertNavLinkVisible(page, "Parked Messages");
await AssertNavLinkHidden(page, "Audit Log");
await AssertNavLinkHidden(page, "Configuration Audit Log");
}
// ── Multi-role user (Admin + Design + Deployment) ───────────────