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

Record page: secondary requests (aggregate, related entities) require login #2362

Open
jrchudy opened this issue Oct 11, 2023 · 0 comments
Open
Labels
discussion required requires a discussion before moving forward

Comments

@jrchudy
Copy link
Member

jrchudy commented Oct 11, 2023

This issue occurred at this link in facebase. When going to a record page, the user is given a login dialog. For users with less privileges, after logging in, they are given a Forbidden error dialog.

Some conversation points that came up while discussing this:

  • should this warning/error be handled differently for different cases
    • aggregate in main section
    • inline related table main data
    • aggregate in inline related table
    • related table main data
    • aggregate in related table
  • should this be a warning instead? no error?
  • OR we can supply a degraded UX similar to what we do for facets when the data can't be fetched (or times out)
    • show a "warning" icon in the column header or related table header with a tooltip explaining the issue better
    • acts like a scoped warning to the location of the error instead of a page warning
  • Curation status has a similar error scenario when a user is logged in then gets logged out
    • they are given a 409 error since anonymous users can't see curation status
    • the model the app thinks is different than what an anonymous user can see
@RFSH RFSH changed the title Record page requires login Record page: secondary requests (aggregate, related entities) require login Feb 1, 2024
@RFSH RFSH added the discussion required requires a discussion before moving forward label Feb 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
discussion required requires a discussion before moving forward
Projects
None yet
Development

No branches or pull requests

2 participants