Skip to content

Conversation

@jackmcdade
Copy link
Member

@jackmcdade jackmcdade commented Jan 30, 2026

Note

Low Risk
UI-only changes that mutate in-memory permission checked flags; main risk is incorrect bulk selection state propagation in nested permission trees.

Overview
Adds Check/Uncheck All controls to the role publish form: a global toggle next to Save and per-permission-group toggles in each group header, with recursive logic to compute “all checked” and apply bulk selection across nested permissions.

Updates PermissionTree.vue to watch initialPermissions and keep its local permissions state in sync when parents mutate the permission structure. Adds new checkbox and checkbox-uncheck SVG icons used by the new buttons.

Written by Cursor Bugbot for commit bea8698. This will update automatically on new commits. Configure here.

Copy link

@cursor cursor bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Cursor Bugbot has reviewed your changes and found 1 potential issue.

Bugbot Autofix is OFF. To automatically fix reported issues with Cloud Agents, enable Autofix in the Cursor dashboard.

@jackmcdade jackmcdade merged commit a59e572 into 6.x Jan 30, 2026
13 checks passed
@jackmcdade jackmcdade deleted the permissions-check-all-button branch January 30, 2026 18:23
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.

3 participants