-
Notifications
You must be signed in to change notification settings - Fork 20
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Filament Shield implementation error #13
Comments
Well it has no specific implementation for the filament shield plugin, but I'll look into it! |
I'm trying to use it with Filament Shield too but my error is when I try to edit a Role:
|
@robertosanval, I haven't used Shield yet, but I'll try to move it up on my list to check it out. If you have any ideas, feel free to make a pull request. |
i'm having the same issue: debugbar gives me little more info:
|
Also i see in the ResourceLockResource.php that there are only to policies: canViewAny() and canDeleteAny() instead of the default filament permission_prefixes. for now i disabled the resource permissions in filament-shield.php config file by adding ResourceLockResource to the resources array:
it also helps mitigating this issue: #5 |
perhaps it has to do with the package name and the way Shield package identifies the permission name: see: https://github.com/bezhanSalleh/filament-shield?tab=readme-ov-file#configure-permission-identifier the default permission identifier does some things with the string "Resource" |
yes the package namespace of ResourceLock in combination with the FilamentShield resource suffix for permissions makes this go wrong You can fix it by adding this code to your AppServiceProvider.php boot() method:
This makes all your Resource permissions structure in the database column look like this:
|
Thank you for figuring this out @eelco2k ! |
@Jehizkia Thanks! |
Hi.
Does the plugin has Filament Shield implementation?
https://github.com/bezhanSalleh/filament-shield
It thows an error
The text was updated successfully, but these errors were encountered: