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

#190 - Permissions #353

Merged
merged 13 commits into from
Sep 25, 2023
Merged

#190 - Permissions #353

merged 13 commits into from
Sep 25, 2023

Conversation

mlencki
Copy link
Member

@mlencki mlencki commented Sep 21, 2023

This should close #190

New edit permissions form is available through users section

image

image

Instead depending on roles in authorization, we're now utilizing permissions using laravel-permission package.

Old roles are still available in user creation & edit forms. They're used to initialize the default set of permissions for each role, which can later be modified in the new edit permissions form.

php artisan toby:sync-permissions should be run after deployment to migrate current roles to permissions and php artisan db:seed --class=PermissionsSeeder to seed permissions.

@mlencki mlencki marked this pull request as ready for review September 21, 2023 13:16
@mlencki mlencki requested a review from a team as a code owner September 21, 2023 13:16
Copy link
Member

@EwelinaSkrzypacz EwelinaSkrzypacz left a comment

Choose a reason for hiding this comment

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

Please add in pr description what commands must be invoked for the application to work.

database/seeders/DemoSeeder.php Outdated Show resolved Hide resolved
@EwelinaSkrzypacz EwelinaSkrzypacz temporarily deployed to beta September 25, 2023 11:42 — with GitHub Actions Inactive
@mlencki mlencki merged commit 6a0e723 into main Sep 25, 2023
8 of 10 checks passed
@mlencki mlencki deleted the #190-permissions branch September 25, 2023 11:57
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.

Refactor roles
2 participants