Freegle / Iznik / 22940 / 22940
71%
master: 72%

LAST BUILD BRANCH: feature/ai-support-helper-rebuild
DEFAULT BRANCH: master
Ran
Files 456
Run time 18s
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst
coverage: 68.067% (-0.03%) from 68.101%
22940

Pull #851

circleci

edwh 
fix(admin): restore sent-admin archive in Previous tab + stop cross-group leak

ModTools "Previous" admin tab showed only stale, approved-but-never-sent
admins (mods saw a years-old Christmas message as the newest), and Admin/
Support users saw pending admins for groups they aren't on.

Root cause: the V2 Go ListAdmins query filtered `complete IS NULL`, which
excludes every *sent* admin (those have `complete` set) - the opposite of
what the Previous archive needs. And the Admin/Support branch applied no
group scoping at all, so with no group selected it swept every group's
admins into the Pending tab.

Fix:
- Drop the `complete IS NULL` filter so sent admins appear again, matching
  V1 Admin::listForGroup (all admins for the group, newest first).
- Scope the default listing to the caller's active mod groups for everyone
  (matching V1 Admin::listPending); Admin/Support may still view a specific
  group's history when they explicitly request a groupid.

Tests: TestListAdminsIncludesCompleted (completed admins are returned);
TestListAdminsSystemAdmin reworked to assert no cross-group leak with no
group, and access to a specific requested group.

## Discourse

https://discourse.ilovefreegle.org/t/admin-messages/9816/4

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Pull Request #851: fix(admin): restore sent-admin archive in Previous tab + stop cross-group leak

24602 of 36144 relevant lines covered (68.07%)

16.97 hits per line

Source Files on job laravel - 22940