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

SearchKit - Fix permission to access default display and download spreadsheets #21977

Merged
merged 1 commit into from
Nov 5, 2021

Conversation

colemanw
Copy link
Member

@colemanw colemanw commented Nov 4, 2021

Overview

Followup to #21929 to ensure all users who should have access to view search displays will be able to do so.

Before

Default display and spreadsheet download were restricted to administrators; this was an oversight.

After

Fixed.

Technical Details

Intentionally setting these gatekeeper permissions fairly open as all of these actions perform stricter permission checks internally.

…eadsheets

Intentionally setting these gatekeeper permissions fairly open as all of these actions
perform stricter permission checks internally.
@civibot
Copy link

civibot bot commented Nov 4, 2021

(Standard links)

@civibot civibot bot added the 5.44 label Nov 4, 2021
@seamuslee001
Copy link
Contributor

This seems fine to me

@seamuslee001 seamuslee001 merged commit 91c65a7 into civicrm:5.44 Nov 5, 2021
@seamuslee001 seamuslee001 deleted the fixSearchKitPermissions branch November 5, 2021 03:17
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.

2 participants