Skip to content

TIGER-335 batch 2: _isAdmin()→_isAllowed() in analytics/code/translations/profile/access - #351

Merged
WebTigers merged 1 commit into
mainfrom
feat/tiger-335-batch2-admin-modules
Oct 21, 2026
Merged

WebTigers merged 1 commit into
mainfrom
feat/tiger-335-batch2-admin-modules

Conversation

@WebTigers

Copy link
Copy Markdown
Owner

Batch 2 of the TIGER-335 per-module migration. Mechanical, behavior-preserving conversion off the deprecated resource-level _isAdmin() in five more blanket-granted modules:

  • analytics (TIGER-354)
  • code (TIGER-345) — incl. the datatable can_edit/can_delete privilege flags (already the correct shape — rename only)
  • translations (TIGER-346)
  • profile (TIGER-340) — the admin Org* services
  • access (TIGER-339) — incl. the User/Org datatable flags

Each service is a wildcard-privilege grant to its role, so _isAllowed('<method>') is behavior-identical; no acl.ini changes. Tests green: 206.

🤖 Generated with Claude Code

https://claude.ai/code/session_01ASauLLscjqdsNqBNsx2Typ

…ations/profile/access (v1.33.0)

Mechanical, behavior-preserving conversion off the deprecated resource-level comparer in five more
blanket-granted modules (incl. the with-privilege datatable can_edit/can_delete flags in code + access,
which were already the correct shape — rename only). No acl.ini changes. Tests green: 206.

Closes TIGER-339, TIGER-340, TIGER-345, TIGER-346, TIGER-354.

Co-Authored-By: Claude Opus 4.8 <noreply@anthropic.com>
Claude-Session: https://claude.ai/code/session_01ASauLLscjqdsNqBNsx2Typ
@WebTigers
WebTigers merged commit 1cd1785 into main Oct 21, 2026
14 checks passed
@WebTigers
WebTigers deleted the feat/tiger-335-batch2-admin-modules branch October 21, 2026 13:51
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