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

Added reporting-only access for selectlists #15272

Merged
merged 1 commit into from
Aug 12, 2024

Conversation

snipe
Copy link
Owner

@snipe snipe commented Aug 12, 2024

While this is an unusual case, this PR adds the ability for users with only report-view capabilities to see select lists. Previously, if a user only had report-view and no ability to edit/create anything else, the select lists in the custom report would not load properly.

Copy link

what-the-diff bot commented Aug 12, 2024

PR Summary

  • Enhanced Access Control in AuthServiceProvider
    The AuthServiceProvider.php file was updated with an additional authentication rule. Now, along with the existing conditions, a user is also checked if they have the ability to view reports, ensuring only authorized persons can access this feature.

@snipe snipe merged commit e6106aa into develop Aug 12, 2024
5 of 9 checks passed
@snipe snipe deleted the fixes/added_selectlist_permission_for_reports branch August 12, 2024 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant