Skip to content
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

Remove "RolePermission" model and table #4054

Open
ssddanbrown opened this issue Feb 20, 2023 · 0 comments
Open

Remove "RolePermission" model and table #4054

ssddanbrown opened this issue Feb 20, 2023 · 0 comments

Comments

@ssddanbrown
Copy link
Member

The permissions available to roles are relatively fixed and not user controllable.
They are often used by name (string) within the codebase for checking permissions.

It would probably be better just to maintain these as code rather than via the database, so changes don't need migrations and so we can be programatic references to permissions within code.

These would probably work best as an Enum so waiting until we have PHP8.1 minimum support may be convenient since there's otherwise no specific rush for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

1 participant