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

Provide an indication that staff are viewing a restricted item #11257

Closed
gestchild opened this issue Aug 13, 2024 · 10 comments
Closed

Provide an indication that staff are viewing a restricted item #11257

gestchild opened this issue Aug 13, 2024 · 10 comments

Comments

@gestchild
Copy link
Contributor

gestchild commented Aug 13, 2024

Relates to wellcomecollection/platform#5761

Once staff are able to view restricted items it would be useful to provide an indication on the page when they are doing this.

In the past there has been confusion when staff are able to see things that members of the public are not and we should make it more obvious why this is the case.

This will be need on the items page, e.g. https://wellcomecollection.org/works/pzmyhgsk/items

and possibly the work page https://wellcomecollection.org/works/pzmyhgsk/

Normally, the work page won't show a link for restricted items, but will do for staff who are logged in

  • Because not all iiif manifests have v2 of the external service and (v1 doesn't work) for granting access - we should adapt the message to notify the user that the iiif manifest needs regenerating in order to be able to see it. We should do this for logged in users with a role of 'StaffWithRestricted' when the external service id is pointing to v1, i.e. https://iiif.wellcomecollection.org/auth/restrictedlogin
@dana-saur dana-saur self-assigned this Aug 14, 2024
@dana-saur dana-saur removed their assignment Aug 15, 2024
@gestchild
Copy link
Contributor Author

@rcantin-w
Copy link
Contributor

@dana-saur @LaurenFBaily just checking if the exact copy in the mockup is good to go forward with?

@dana-saur
Copy link

@rcantin-w I sent this updated copy to Gareth after last week's review because it's a little clearer. I'll update the designs.

Restricted item This page is hidden from the public and can only be viewed by some staff.

@dana-saur
Copy link

I also like the 'i' icon that Gareth used in the review session so happy to keep that. I can't see that icon in Figma so I'll add it from cardigan

@rcantin-w rcantin-w transferred this issue from wellcomecollection/platform Oct 10, 2024
@rcantin-w rcantin-w moved this from Backlog to Next in Digital experience Oct 10, 2024
@rcantin-w rcantin-w self-assigned this Oct 14, 2024
@rcantin-w rcantin-w moved this from Next to In progress in Digital experience Oct 14, 2024
@dana-saur
Copy link

Had a chat with Raph, Gareth and Ashley. We agreed that when something is fully restricted, the most appropriate word to describe is item.

@rcantin-w updated copy below:

Item viewer

Restricted item
This item is hidden from the public.

image

Works page

Restricted item
This item is hidden from the public and can only be viewed by staff.

image

@dana-saur
Copy link

Also split out the partially restricted indicator to separate ticket: #11278

And updating the existing public facing restricted material copy for consistency with the staff copy changes above: #11277

@rcantin-w
Copy link
Contributor

Thanks for this @dana-saur ! Pushing the changes now.

@rcantin-w
Copy link
Contributor

This is now in production.

@github-project-automation github-project-automation bot moved this from Ready for review to Done in Digital experience Oct 17, 2024
@rcantin-w
Copy link
Contributor

Slack conversation regarding this for legacy purposes

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Archive
Development

No branches or pull requests

3 participants