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

Test authentication for UCLA #28

Open
dbietila opened this issue Dec 12, 2023 · 6 comments
Open

Test authentication for UCLA #28

dbietila opened this issue Dec 12, 2023 · 6 comments
Assignees

Comments

@dbietila
Copy link

dbietila commented Dec 12, 2023

In the Access menu:

  • "Log in" should appear, unless the user already is logged in (has a Shib session)
  • "Request an Account" should appear unless the user is logged in.
  • If the user is logged in, a field should display saying "Logged in as "CNetID".
  • "Request Access to this Series" should appear if the user is logged in and on a Series or Item page.

Here are some things to check for with requesting authentication and access to Panopto recordings:

  • Each series should display a tag indicating its access level "Open", "Account Required" (formerly "Campus"), and "By Request" (formerly "Restricted")

When testing Panopto player, remember to check your account privileges in Panopto
The Panopto player should load:

  • For Open items, always
    • Download icon should be available
  • For Account Required items, the player should load if the user is logged in
    • Download icon should not be available (streaming only)
  • For By Request items, the player should load only if the user is logged in and has permission for the Series
    • Download icon should not be available (streaming only)
@vitorgomateus
Copy link
Collaborator

"Request Access to this Series" should appear if the user is logged in and on a Series or Item page.

When a series requires access, but the user is not logged in, there is currently the same button "Request Access to this Series" but it triggers an alert informing of the need to an account.
I can change it to have the message show directly under access instead of the button.
The current solution seems to me visually cleaner.
@dbietila can we keep this as it is?

@dbietila
Copy link
Author

I think the current behavior is okay.

@vitorgomateus
Copy link
Collaborator

"Request Access to this Series" should appear if the user is logged in and on a Series or Item page.

Currently (following the lates changes with #32) this link only appears if the series is restricted, has an item with a panopto link. At an item level view, if the item itself doesn't have a panopto link, the link won't show. User auth status is detected at the request access page itself.

@vitorgomateus
Copy link
Collaborator

All the Series that I found with "Campus" status have no panopto link in their sound type items.

@vitorgomateus
Copy link
Collaborator

@bufordrat
Copy link
Collaborator

Yeah, we had a big back and forth on this. The TDLR:

  • It's the right path; the problem is that Okta is not turned on on ucla.lib.
  • I started a Slack channel last week asking Fred to turn it on.
  • During the D meeting yesterday, Charles said that Okta should not be turned on on ucla.lib, because the stakeholders just changed their minds about what the hostname of the production UCLA site should be, which means Fred can't really send a "turn on Okta" request to IT services until we've settled on the hostname we're going to use.

I'm still unclear on what all of the above means logistically re: our ability to test that the "soft-launch" production site is working. I would follow up with @dbietila to see what he recommends.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants