Skip to content

Conversation

@lancepioch
Copy link
Member

Screen.Recording.2026-01-30.at.3.35.47.PM.mov

@lancepioch lancepioch self-assigned this Jan 30, 2026
@coderabbitai
Copy link

coderabbitai bot commented Jan 30, 2026

📝 Walkthrough

Walkthrough

Adds a "Set All Permissions" toggle group to the API Key form that allows users to quickly configure permissions across all resources. The feature includes language strings for UI labels and state synchronization logic to update individual per-resource permission toggles.

Changes

Cohort / File(s) Summary
API Key Form Enhancement
app/Filament/Admin/Resources/ApiKeys/ApiKeyResource.php
Introduces a permissionAll toggle group with none, read, and read_write options. Uses afterStateUpdated to propagate changes to individual per-resource toggles. Caches permission list to reduce redundant calls.
Localization Strings
lang/en/admin/apikey.php
Adds two language keys ('all' and 'all_description') to support the new Set All Permissions feature in the permissions section.

Sequence Diagram

sequenceDiagram
    actor User
    participant Form as API Key Form
    participant ToggleGroup as "Set All Permissions<br/>Toggle Group"
    participant StateHandler as "afterStateUpdated<br/>Handler"
    participant PerResourceToggles as "Per-Resource<br/>Permission Toggles"

    User->>ToggleGroup: Select permission level (none/read/read_write)
    ToggleGroup->>Form: Emit state change
    Form->>StateHandler: Trigger afterStateUpdated callback
    StateHandler->>StateHandler: Iterate through permission list
    StateHandler->>PerResourceToggles: Update each resource toggle to selected level
    PerResourceToggles->>Form: Reflect changes in form state
    Form->>User: Display updated permission toggles
Loading
🚥 Pre-merge checks | ✅ 3
✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and accurately summarizes the main change: adding functionality to toggle all API token permissions at once.
Description check ✅ Passed The description references a video asset demonstrating the feature but provides minimal textual context; however, it relates to the changeset's purpose of toggling all permissions at once.
Docstring Coverage ✅ Passed Docstring coverage is 100.00% which is sufficient. The required threshold is 80.00%.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

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.

2 participants