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

Add import listing for admins #26316

Merged
merged 1 commit into from
Jun 10, 2023
Merged

Conversation

eileenmcnaughton
Copy link
Contributor

Overview

@colemanw is is possible to have a search display only show for some users? This search is intended to allow the admin users with permissions to see OTHER people's imports a list of all imports - but for users without that permission this is going to only show their own imports - which the 'My Imports' search already does... I can make it for some users in the menu I guess...

Before

What is the old user-interface or technical-contract (as appropriate)?
For optimal clarity, include a concrete example such as a screenshot, GIF (LICEcap, SilentCast), or code-snippet.

After

What changed? What is new old user-interface or technical-contract?
For optimal clarity, include a concrete example such as a screenshot, GIF (LICEcap, SilentCast), or code-snippet.

Technical Details

If the PR involves technical details/changes/considerations which would not be manifest to a casual developer skimming the above sections, please describe the details here.

Comments

Anything else you would like the reviewer to note

@civibot
Copy link

civibot bot commented May 24, 2023

(Standard links)

@civibot civibot bot added the master label May 24, 2023
@colemanw
Copy link
Member

@eileenmcnaughton yes you could set a permission on the Afform you embed this display on

@eileenmcnaughton
Copy link
Contributor Author

@colemanw I think this is probably mergeable - it will show up in packages searchs only with this - so when I add to the menu via afform I can permission it

@colemanw colemanw merged commit cf36e07 into civicrm:master Jun 10, 2023
@colemanw colemanw deleted the import_admin branch June 10, 2023 14:03
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