Skip to content

feat(marketplace): access audit list and per-user block controls#8507

Open
LWS49 wants to merge 1 commit into
lws49/feat-marketplace-pr6c-allowlistfrom
lws49/feat-marketplace-pr6d-audit
Open

feat(marketplace): access audit list and per-user block controls#8507
LWS49 wants to merge 1 commit into
lws49/feat-marketplace-pr6c-allowlistfrom
lws49/feat-marketplace-pr6d-audit

Conversation

@LWS49

@LWS49 LWS49 commented Jul 22, 2026

Copy link
Copy Markdown
Collaborator

Summary

Adds the access audit section beneath the allow-list: a list of everyone who currently has marketplace access, why they are eligible, and which rule or rules grant it. The list is filterable by status and by granting rule, each row carries block and re-enable controls, and a separate section surfaces dormant blocks - blocks that no rule currently matches.

Design decisions

  • Blocking is exposed per person from the audit list rather than on its own screen - the decision to block is made while reading who has access, not in isolation.
  • Dormant blocks are surfaced rather than hidden - such a block denies nothing today, but it takes effect again the moment a rule starts matching that person, so an admin has to be able to see and clear it.
  • Rules that currently grant access to nobody are flagged in place rather than auto-removed - an admin may deliberately add a rule ahead of the people it will match.

Regression prevention

Covers: audit list rendering with eligibility and granting-rule annotations, system administrators listed without any matching rule, rows present only because the person is blocked, the status and granting-rule filters plus clear-all, block and re-enable including failure toasts, the dormant blocks section, and the summary and filtered count lines.

@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6c-allowlist branch 2 times, most recently from c02489f to dcd6d34 Compare July 22, 2026 05:41
@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6d-audit branch from b466380 to 0f0d6fc Compare July 22, 2026 05:43
@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6c-allowlist branch from dcd6d34 to 1b71da5 Compare July 22, 2026 05:54
@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6d-audit branch from 0f0d6fc to 1d045a4 Compare July 22, 2026 05:59
@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6c-allowlist branch from 1b71da5 to 0234290 Compare July 22, 2026 06:01
@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6d-audit branch from 1d045a4 to 778cb35 Compare July 22, 2026 06:05
Add the access audit section to the allow-list page: an audit list of everyone
with effective access (filterable by status and granting rule), block/unblock
controls per user, and the rule-match counts that flag allow-list rules which
currently grant access to nobody.
@LWS49
LWS49 force-pushed the lws49/feat-marketplace-pr6d-audit branch from 778cb35 to a5d99e5 Compare July 22, 2026 06:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant