fix: a report's details keep the space before a note, and the Reports menu test opens the row it means (M503 #5606)
release / govulncheck (push) Successful in 19s
release / web (push) Successful in 1m38s
release / go (push) Successful in 1m49s
release / android (push) Successful in 5m22s
release / integration (push) Successful in 5m27s
release / Build signed APK (releases and dev) (push) Successful in 5m30s
release / Attach APK to the Release (tag releases only) (push) Skipped
release / Build + push container image (push) Successful in 26s
release / Verify release artifacts (tag releases only) (push) Skipped

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This commit is contained in:
2026-10-11 10:57:02 -04:00
co-authored by Claude Opus 5.5
parent 89a2653995
commit 0cc4bc686a
2 changed files with 4 additions and 3 deletions
+2 -2
View File
@@ -254,14 +254,14 @@
{#each r.reports as rep, i (rep.user_id + i)}
<li>
<span class="text-text-primary">{rep.username}</span> flagged it ·
{REASON_LABELS[rep.reason] ?? rep.reason}{#if rep.notes} · <span class="italic">{rep.notes}</span>{/if}
{REASON_LABELS[rep.reason] ?? rep.reason}{#if rep.notes}{' · '}<span class="italic">{rep.notes}</span>{/if}
· {relativeTime(rep.created_at)}
</li>
{/each}
{#each r.errors as e (e.id)}
<li>
Failed to play for <span class="text-text-primary">{e.username}</span> ·
{ERROR_LABELS[e.kind] ?? e.kind}{#if e.detail} · {e.detail}{/if} · {relativeTime(e.occurred_at)}
{ERROR_LABELS[e.kind] ?? e.kind}{#if e.detail}{` · ${e.detail}`}{/if} · {relativeTime(e.occurred_at)}
</li>
{/each}
{#if r.file_path}
+2 -1
View File
@@ -103,7 +103,8 @@ describe('/admin/reports', () => {
setup([flagged], [failure('e2', 't2', 'Intro', 2)]);
expect(screen.getByRole('button', { name: /delete via lidarr for roygbiv/i })).toBeInTheDocument();
expect(screen.queryByRole('button', { name: /delete via lidarr for intro/i })).toBeNull();
await fireEvent.click(screen.getAllByRole('button', { name: /more actions/i })[0]);
const roygbiv = screen.getAllByTestId('report-row').find((r) => r.textContent?.includes('Roygbiv'))!;
await fireEvent.click(within(roygbiv).getByRole('button', { name: /more actions/i }));
expect(within(screen.getByRole('menu')).getByText('Delete via Lidarr')).toBeInTheDocument();
});