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

Pub workflow #881

Merged
merged 7 commits into from
Dec 20, 2024
Merged

Pub workflow #881

merged 7 commits into from
Dec 20, 2024

Conversation

vincentauger
Copy link
Member

This pull request introduces several changes to the permissions and roles system, updates various controllers and policies, and includes some dependency updates. The most significant changes are detailed below:

Permissions and Roles Enhancements:

  • Added new user permissions: PUBLISH_INTERNAL_REPORTS and UPDATE_PUBLICATIONS in UserPermission enum (app/Enums/Permissions/UserPermission.php).
  • Updated UserRole enum to include new roles (EDITOR and CHIEF_EDITOR), and restructured the permissions method to support these roles (app/Enums/Permissions/UserRole.php). [1] [2]

Controllers and Policies Updates:

  • Modified PublicationController to check user permissions for viewing and updating publications, and added a new publish method in PublicationPolicy to handle publication permissions (app/Http/Controllers/PublicationController.php, app/Policies/PublicationPolicy.php). [1] [2] [3] [4] [5] [6] [7]
  • Updated AuthorController to use Gate::authorize for authorization checks and corrected the author creation method (app/Http/Controllers/AuthorController.php).

Filament Resource Enhancements:

  • Enhanced UserResource to display role labels and colors dynamically using the UserRole enum (app/Filament/Resources/UserResource.php). [1] [2] [3]

Dependency and Configuration Updates:

  • Updated Laravel framework dependency to version ^11.36.1 in composer.json (composer.json).
  • Moved MediaPolicy registration from AuthServiceProvider to AppServiceProvider and added a new method to configure gates (app/Providers/AppServiceProvider.php, app/Providers/AuthServiceProvider.php). [1] [2] [3] [4]

These changes collectively improve the system's role and permission management, enhance authorization checks, and update dependencies for better performance and security.

Copy link

✔️ All Checks Passed!

🛡️ 0 Security Vulnerabilities
🚀 0 Performance Issues
🐛 0 Reliability Issues

You may view the full report here.

@vincentauger vincentauger merged commit a76c28d into main Dec 20, 2024
4 checks passed
@vincentauger vincentauger deleted the pub-workflow branch December 23, 2024 18:09
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.

1 participant