Skip to content

Normal users can view old cohorts#1369

Merged
mbarton merged 27 commits intolivefrom
mbarton/view-old-cohorts-2
Apr 21, 2026
Merged

Normal users can view old cohorts#1369
mbarton merged 27 commits intolivefrom
mbarton/view-old-cohorts-2

Conversation

@mbarton
Copy link
Copy Markdown
Member

@mbarton mbarton commented Apr 20, 2026

Closes #1268. Respin of #1346 with Claude doing the heavy lifting.

Audit team members can already view and edit children in closed cohorts. This PR adds support for normal users to view children those cohorts without being able to edit them.

@mbarton mbarton marked this pull request as draft April 20, 2026 18:18
@mbarton mbarton changed the title View old cohorts Normal users can view old cohorts Apr 20, 2026
@mbarton mbarton marked this pull request as ready for review April 21, 2026 09:43
mbarton and others added 6 commits April 21, 2026 10:51
- Added is_active and email_confirmed checks for non-superusers
- Added active=True filter on Organisation queries
- Added guard against None organisation_employer (prevents AttributeError)
…closed cohorts.

OrganisationEmployer is the trust relationship here.
@mbarton mbarton merged commit 0d20709 into live Apr 21, 2026
1 check passed
@mbarton mbarton deleted the mbarton/view-old-cohorts-2 branch April 21, 2026 14:17
@rcpch-prout
Copy link
Copy Markdown

rcpch-prout Bot commented Apr 21, 2026

Seen on STAGING, LIVE (merged by @mbarton 10 minutes and 47 seconds ago) Please check your changes!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Users not being able to view Cohort 6 entries

1 participant