Add link to Collections AI site

yalelibrary / YUL-DC

Preliminary issue tracking for Yale University Libraries Digital Collections project

3 stars 0 forks source link

Add link to Collections AI site #2931

Closed mikeapp closed 2 months ago

mikeapp commented 2 months ago

Story

We would like to show a link to the Collections AI tool but only to users who have been added to the correct group in our authentication system.

Acceptance

[x] To logged-in users in the correct group, a link appears beneath the Universal Viewer and full text display area.
- [x] The link does not appear on OwP objects
[x] The link does not appear to other users.

mikeapp commented 2 months ago

WIP PR: https://github.com/yalelibrary/yul-dc-blacklight/pull/1051

sshetenhelm commented 2 months ago

@mikeapp Would it be possible to ensure this message displays below any current full-text? Or is it only for object with Full Text = None ?

mikeapp commented 2 months ago

I made the change as requested and adjusted the width to follow the metadata block. I'd slightly prefer to have it directly under the UV and track the width of the UV, but happy either way since very few people will see it at all.

You can see that the CSS widths and breakpoints for the UV, the full text controls, and the metadata block aren't consistent across all screen widths. Here are some examples from different widths, sometimes the metadata is wider than UV, sometimes not, etc.:

I don't know whether this was intended.

sshetenhelm commented 2 months ago

Well huh. I mean, if it looks better the other way, feel free to put it directly below the UV.

I'll make a separate ticket for looking into the metadata and UV container widths.

mikeapp commented 2 months ago

PR here: https://github.com/yalelibrary/yul-dc-blacklight/pull/1051

Tests verify this doesn't show up to normal users

K8Sewell commented 2 months ago

Deployed to Test with release v1.64.3

K8Sewell commented 2 months ago

As a not logged in user and as logged in user I don't see the AI link on fulltext or non fulltext YCO objects. @mikeapp is there additional configuration that needs to happen to add me to the cognito group that has permission to view the AI link? Perhaps I don't have the correct permission and the groups are configured in the cognito dashboard.

mikeapp commented 2 months ago

Looks good to me on test

mikeapp commented 2 months ago

On UAT, the box w/link should not appear to anyone but a few folks (me and DevOps).

In production, I've removed Lauren's access, so she can verify that it does not appear for normal users. Summer is in the AI user group in Cognito and should be able to see the link.

mikeapp commented 2 months ago

Looks good to me on UAT