160 lines
7.4 KiB
HTML
160 lines
7.4 KiB
HTML
<!DOCTYPE html>
|
|
<html lang="en">
|
|
<head>
|
|
<meta charset="UTF-8">
|
|
<meta name="viewport" content="width=device-width, initial-scale=1.0">
|
|
<title>GuruRMM — Small Fixes Show Notes</title>
|
|
<style>
|
|
:root {
|
|
--bg: #0f1115;
|
|
--panel: #171a21;
|
|
--panel-2: #1d2129;
|
|
--border: #2a2f3a;
|
|
--text: #e6e9ef;
|
|
--muted: #9aa3b2;
|
|
--accent: #4f9cff;
|
|
--p1: #ff5d5d;
|
|
--med: #ffb84d;
|
|
--low: #5ad19a;
|
|
--mono: ui-monospace, "Cascadia Code", "SF Mono", Consolas, monospace;
|
|
}
|
|
* { box-sizing: border-box; }
|
|
body {
|
|
margin: 0;
|
|
font-family: -apple-system, "Segoe UI", Roboto, Helvetica, Arial, sans-serif;
|
|
background: var(--bg);
|
|
color: var(--text);
|
|
line-height: 1.55;
|
|
}
|
|
.wrap { max-width: 960px; margin: 0 auto; padding: 48px 24px 96px; }
|
|
header { border-bottom: 1px solid var(--border); padding-bottom: 24px; margin-bottom: 32px; }
|
|
h1 { font-size: 28px; margin: 0 0 6px; letter-spacing: -0.4px; }
|
|
.sub { color: var(--muted); font-size: 14px; }
|
|
.meta { margin-top: 14px; display: flex; gap: 18px; flex-wrap: wrap; font-size: 13px; color: var(--muted); }
|
|
.meta b { color: var(--text); font-weight: 600; }
|
|
h2 { font-size: 16px; text-transform: uppercase; letter-spacing: 1px; color: var(--accent); margin: 40px 0 14px; }
|
|
table { width: 100%; border-collapse: collapse; background: var(--panel); border: 1px solid var(--border); border-radius: 10px; overflow: hidden; }
|
|
th, td { text-align: left; padding: 12px 14px; border-bottom: 1px solid var(--border); vertical-align: top; font-size: 14px; }
|
|
th { background: var(--panel-2); font-size: 11px; text-transform: uppercase; letter-spacing: 0.8px; color: var(--muted); }
|
|
tr:last-child td { border-bottom: none; }
|
|
td.id { font-family: var(--mono); font-size: 13px; white-space: nowrap; color: var(--accent); }
|
|
td.size { font-family: var(--mono); font-size: 13px; color: var(--muted); white-space: nowrap; }
|
|
.pill { display: inline-block; padding: 2px 9px; border-radius: 999px; font-size: 11px; font-weight: 700; letter-spacing: 0.4px; }
|
|
.pill.p1 { background: rgba(255,93,93,0.15); color: var(--p1); border: 1px solid rgba(255,93,93,0.4); }
|
|
.pill.med { background: rgba(255,184,77,0.13); color: var(--med); border: 1px solid rgba(255,184,77,0.4); }
|
|
.pill.low { background: rgba(90,209,154,0.13); color: var(--low); border: 1px solid rgba(90,209,154,0.4); }
|
|
.rec { background: linear-gradient(180deg, var(--panel-2), var(--panel)); border: 1px solid var(--border); border-left: 3px solid var(--accent); border-radius: 10px; padding: 18px 20px; margin-top: 16px; }
|
|
.rec b { color: var(--accent); }
|
|
code { font-family: var(--mono); font-size: 12.5px; background: var(--panel-2); padding: 1px 6px; border-radius: 5px; color: #cdd6e4; }
|
|
.note { color: var(--muted); font-size: 13px; margin-top: 28px; padding-top: 18px; border-top: 1px solid var(--border); }
|
|
footer { margin-top: 48px; color: var(--muted); font-size: 12px; text-align: center; }
|
|
</style>
|
|
</head>
|
|
<body>
|
|
<div class="wrap">
|
|
<header>
|
|
<h1>GuruRMM — Small Fixes</h1>
|
|
<div class="sub">Show notes: quick, well-scoped wins pulled from the roadmap + UI gaps</div>
|
|
<div class="meta">
|
|
<span><b>Date:</b> 2026-05-30</span>
|
|
<span><b>Sources:</b> FEATURE_ROADMAP.md, UI_GAPS.md</span>
|
|
<span><b>Scope:</b> sub-day fixes only</span>
|
|
</div>
|
|
</header>
|
|
|
|
<h2>Frontend fixes (React / TS — no server change)</h2>
|
|
<table>
|
|
<thead><tr><th>ID</th><th>Fix</th><th>Size</th><th>Pri</th></tr></thead>
|
|
<tbody>
|
|
<tr>
|
|
<td class="id">BUG-014</td>
|
|
<td>Site Detail page has no agent search box — add <code>agentSearch</code> state + filter. Every other list page already has this.</td>
|
|
<td class="size">~30 min</td>
|
|
<td><span class="pill p1">P1</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">Error boundary</td>
|
|
<td>No <code>ErrorBoundary</code> anywhere — one component throw whites out the whole app. Wrap <code><AppRoutes></code> with fallback + reload.</td>
|
|
<td class="size">~30 min</td>
|
|
<td><span class="pill med">MED</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">BUG-009</td>
|
|
<td><code>Logs.tsx</code> — 3 <code>useQuery</code> calls with no <code>isError</code> handling (errors look like "no results"). Copy the pattern from <code>Clients.tsx:164</code>.</td>
|
|
<td class="size">~1 hr</td>
|
|
<td><span class="pill med">MED</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">BUG-010</td>
|
|
<td>8 pages still missing <code>isError</code> handling (Alerts, AlertTemplates, Commands, Dashboard, Logs, Settings, Sites, Users) — same pattern.</td>
|
|
<td class="size">~half day</td>
|
|
<td><span class="pill med">MED</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">BUG-011</td>
|
|
<td>14 <code>: any</code> annotations across 6 files — swap to existing typed interfaces / <code>unknown</code> + <code>isAxiosError</code>.</td>
|
|
<td class="size">~1 hr</td>
|
|
<td><span class="pill med">MED</span></td>
|
|
</tr>
|
|
</tbody>
|
|
</table>
|
|
|
|
<h2>Server fixes (Rust)</h2>
|
|
<table>
|
|
<thead><tr><th>ID</th><th>Fix</th><th>Size</th><th>Pri</th></tr></thead>
|
|
<tbody>
|
|
<tr>
|
|
<td class="id">BUG-008</td>
|
|
<td><code>metrics.rs</code> leaks raw <code>e.to_string()</code> DB errors to clients at 5 sites — route through existing <code>internal_err()</code> helper.</td>
|
|
<td class="size">~30 min</td>
|
|
<td><span class="pill med">MED</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">Client row</td>
|
|
<td><code>GET /api/agents/:id</code> hardcodes <code>client_name: None</code>, so AgentDetail's "Client" row never renders — enrich <code>get_agent</code> or drop the field.</td>
|
|
<td class="size">~1 hr</td>
|
|
<td><span class="pill med">MED</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">Fleet counts</td>
|
|
<td>Dashboard recomputes counts client-side; <code>/metrics/summary</code> + <code>/agents/stats</code> exist but are unused — wire them or delete.</td>
|
|
<td class="size">~1 hr</td>
|
|
<td><span class="pill low">LOW</span></td>
|
|
</tr>
|
|
</tbody>
|
|
</table>
|
|
|
|
<h2>Other small ones</h2>
|
|
<table>
|
|
<thead><tr><th>ID</th><th>Fix</th><th>Size</th><th>Pri</th></tr></thead>
|
|
<tbody>
|
|
<tr>
|
|
<td class="id">BUG-015</td>
|
|
<td>Agent missing from Programs & Features — WiX-only edit, add ARP properties + product icon. Spec'd in SPEC-011.</td>
|
|
<td class="size">~1 hr</td>
|
|
<td><span class="pill med">P2</span></td>
|
|
</tr>
|
|
<tr>
|
|
<td class="id">Changelog UI</td>
|
|
<td>Read-only version-history table on AgentDetail. May need one <code>GET</code> endpoint.</td>
|
|
<td class="size">~0.5 day</td>
|
|
<td><span class="pill low">P3</span></td>
|
|
</tr>
|
|
</tbody>
|
|
</table>
|
|
|
|
<div class="rec">
|
|
<b>Recommended quick session:</b> knock out <b>BUG-014 + Error boundary + BUG-008</b> together —
|
|
all three are sub-hour (two frontend, one backend), and BUG-014 is a P1 that Howard reported.
|
|
</div>
|
|
|
|
<div class="note">
|
|
Note: BUG-002 and BUG-006 are already fixed in branch <code>fix/audit-2-remediation</code> but unmerged.
|
|
That branch can be reviewed/merged separately if useful.
|
|
</div>
|
|
|
|
<footer>GuruRMM · Arizona Computer Guru · generated 2026-05-30</footer>
|
|
</div>
|
|
</body>
|
|
</html>
|