Skip to content

Loading failures negatively impact UX #263

@SharonStrats

Description

@SharonStrats

When navigating around my pod, logging out, logging in, viewing things I have in my storage I receive an abrupt message about my resource failing to load. There are different cases when this happens.

Cases where 401/403 Issues arise

  1. Logging Out -
    error: Failed to load https://www.solidcommunity.net/ Fetcher: https://www.solidcommunity.net/ status: 401
    should happen: I would not like to see this error, instead it could say something (not in red), you are logged out. Or it could just show the menu.
  2. Logging in - right away when I log in I get an error
    error: Failed to load https://www.solidcommunity.net/ Fetcher: https://www.solidcommunity.net/ status: 403
    should happen: I should not get an error, maybe it should navigate to the home screen when this happens, although what if you log in while you are looking at another resource you would want to redirect. So logging in from the "home" should be detected somehow and accordingly you should be displayed something specific.
  3. From storage I click on the link icon next to the file name, takes me to the file (I'm logged in already). I receive 2 errors
    errors: Error reading file: Error: Web error: 401 on GET of https://sstratsianis.solidcommunity.net/formtest.ttl and below it Failed to load https://sstratsianis.solidcommunity.net/formtest.ttl Fetcher: https://sstratsianis.solidcommunity.net/formtest.ttl status: 401

when I go through the link below I don't have the same problem, it goes directly to the resource with no failed to load errors. I believe this is the "about" pane. need to find it in solid-panes.
Image

Technical details

  1. Logout We could redirect to a known public page before reload, or skip GotoSubject when URL is app root and unauthenticated. here is the spot
  2. Login Same spot as above. We could at bare root, open the user profile or perhaps storage when logged in, and otherwise skip root fetch.
  3. File Loading Errors After research I believe the only way to solve this is storing cookies or looking through all the links and passing authSession in (need to look at this again and why it's not a good option for production). We can't store auth in cookies because it's not safe. But, we could change the message to be more directed to user. To change the error message style we can do that here

Changes we could make are shown in this PR here

Metadata

Metadata

Assignees

Labels

No labels
No labels

Type

No type

Projects

No projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions