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

Configurable display options—admins should have the option to display only a thumbnail and/or download link instead of the embedded viewer #773

Closed
1 task
Tracked by #903
jillpe opened this issue Sep 8, 2023 · 3 comments

Comments

@jillpe
Copy link

jillpe commented Sep 8, 2023

Summary

Admins can configure only a thumbnail and/or download link in place of the embedded viewer.

The ability to set this via Bulkrax is not included in this estimate

Accepted Criteria

  • As an admin I can configure only a thumbnail and/or download link in place of the embedded viewer for a PDF
@jillpe jillpe mentioned this issue Sep 8, 2023
51 tasks
@kirkkwang
Copy link
Contributor

Possible solution could be setting the PDF file set to private and therefore the viewer should not render. We need to look at what would happen with the thumbnail since the file is private

@ShanaLMoore
Copy link
Contributor

ShanaLMoore commented Sep 15, 2023

QA RESULTS: Pass, but... ⚠️

Accepted Criteria

  • As an admin I can configure only a thumbnail and/or download link in place of the embedded viewer for a PDF

ADMIN ✅

image

788.mp4

NON ADMIN ⚠️

I'm not sure how to test this and may need to rely on the client for the different role types. I tested this with my personal non admin account and I don't have an edit button to see if those check boxes exist for non admin users.

When admin has both checks true:
image

When admin has both checks false:

image

@ShanaLMoore ShanaLMoore moved this from Ready for Development to PALs QA in palni-palci Sep 15, 2023
@ndroark ndroark moved this from PALs QA to Deploy to Production in palni-palci Sep 18, 2023
@ShanaLMoore
Copy link
Contributor

Requires Reindexing prod

@ShanaLMoore ShanaLMoore moved this from Deploy to Production to Client Verification in palni-palci Sep 21, 2023
@ndroark ndroark moved this from Client Verification to Done in palni-palci Oct 2, 2023
@jillpe jillpe closed this as completed Oct 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Done
Development

No branches or pull requests

3 participants