search

RickDBCN / filament-email

Log emails in your Filament project
https://filament-email-demo.marcogermani.it/
MIT License
85 stars 23 forks source link

Add support for Shield #7

Closed malle-pietje closed 1 year ago

malle-pietje commented 1 year ago

Is your feature request related to a problem? Please describe. I always look to control access to resources/pages, Shield is what I'm currently using within Filament.

Describe the solution you'd like The ability to enable support for Shield. For now the plugin doesn't support methods such as usingPage() which make it easy to create an override of the default Page class where we can add the HasPageShield Trait.

Are there better/easier approaches?

RickDBCN commented 1 year ago

Hi @malle-pietje,

If you are using Shield, you could run php artisan shield:generate. The Email page is just a Filament Resource, therefor Shield should automatically generate new policies, so you are able to control who has access to this resource.

Please let me know if this works for you!

malle-pietje commented 1 year ago

Thanks, in that case I need to check Shield which is still allowing access. This my first Filament project after having used several other Laravel packages, so still learning 😉

RickDBCN commented 1 year ago

Thanks, in that case I need to check Shield which is still allowing access. This my first Filament project after having used several other Laravel packages, so still learning 😉

No worries, if you need help, please let me know!