SolidOS / solidos

The operating system for Solid
https://solidos.solidcommunity.net/
MIT License
127 stars 19 forks source link

Webapp does not display storage(s) #86

Closed jeff-zucker closed 3 years ago

jeff-zucker commented 3 years ago

The new storage improvements are not available in the web app. Neither the storage button nor the menu appear.

alain1 .

jeff-zucker commented 3 years ago

Please note : previous to this release (the one currently on :8443) there was a storage button but no menu. The menu we can consider an enhancement and add later, but the disappearance of the storage button and no way to see the multiple storages is a problem.

jeff-zucker commented 3 years ago

I have done some more testing, here's my notes (abbreviating solidcommunity.net as scn)

Using local browse.html from either mashlib 1.78 or 1.79

I get identical behavior from 1.79 so if one of these is without the storage changes, that probably means it is not a new bug. So I suggest we change this to an enhancement and release.

jeff-zucker commented 3 years ago

It's possible that running it local gives different results so it might be useful to have a browse.html available on the test server.

bourgeoa commented 3 years ago

@jeff-zucker I found a situation where it works correctly. Both solidcommunity.net:8443 and solidweb.org are running NSS@5.6.14 with mashlib@1.7.9 image

jeff-zucker commented 3 years ago

Wow, that is really odd from several points of view. First, why would it work there and no where else? Second - why does it show you as visiting and logged in at solidweb but with a storage from solidcommunity,net? What sequence did you follow?

Also please clarify - 1.7.9 is the latest that you reverted from the previous? And 1.7.8 is actually newer?

jeff-zucker commented 3 years ago

I can not reproduce your results, this is with a browse.html on :8443 visiting solidweb.org and logging in on :8443 from there.

jeff-zucker commented 3 years ago

alain2

bourgeoa commented 3 years ago

@jeff-zucker I think issue resolved. The error was cannot load container. replaced :

by :

solidcommunity.net is running with mashlib@1.7.7 and for the time being solidcommunity.net:8443 is running mashlib@1.7.9

mashlib@1.7.8 was wrong and never used in NSS. It is not possible to remove version from npmjs

jeff-zucker commented 3 years ago

When you say resolved, do you mean there is a working version somewhere that I can test? Where?

bourgeoa commented 3 years ago

It will come I tested locally. Shall install on test-server soon.

jeff-zucker commented 3 years ago

OH, I see your commits now. Good Catch! This is exactly the same problem with the IDP dropdown in https://github.com/solid/solidos/issues/88!

bourgeoa commented 3 years ago

Solved with https://github.com/solid/solid-panes/pull/311

bourgeoa commented 3 years ago
  1. As can be seen below dashboard-pane is only displayed when webId and search uri have same origin. There is no regression anymore. But this seem an existing bug. Opened #89
url search uri webId solidcommunity.net solidcommunity.net:8443 localhost.8443
bou bou no-webId default.html default.html default.html + folder-pane bou
bou bou1 no-webId default.html/folder-pane bou1 default.html/folder-pane bou1 default.html + folder-pane bou1
bou bou1 bou-Id default.html/folder-pane bou1 default.html/folder-pane bou1 default.html/folder-pane bou1
bou bou bou-Id dashboard Your Storage dashboard Your Storage dashboard Your Storage
bou bou1 bou1-Id dashboard Your Storage dashboard Your Storage + bou dashboard Your Storage + bou
bou bou bou1-Id default.html/folder-pane bou default.html/folder-pane bou default.html/folder-pane bou
bou1 bou no-webId default.html default.html default.html + folder-pane bou
bou1 bou1 no-webId default.html + folder-pane bou1 default.html + folder-pane bou1 default.html + folder-pane bou1
bou1 bou bou1-Id default.html default.html default.html +folder-pane
bou1 bou1 bou1-Id dashboard dashboard dashboard
bou1 bou bou-Id dashboard Your Storage dashboard Your Storage + bou1 dashboard Your Storage + bou1
bou1 bou1 bou-Id default.html + folder-pane bou1 default.html + folder-pane bou1 default.html + folder-pane bou1

bou : https://bourgeoa.solidcommunity.net/ or https://bourgeoa.solidcommunity.net:8443/ bou1 : https://bourgeoa1.solidcommunity.net/ or https://bourgeoa1.solidcommunity.net:8443/ url : bou/brows.html or bou1/browse.html bou-Id : bou/profile/card#me bou1-Id : bou1/profile/card#me

  1. a problem occured when using an url that is not on a solid server (https://github.io). Issue found shall be resolved too in mashlib > 1.7.10
bourgeoa commented 3 years ago

https://solidos.solidcommunity.net/index.html contains is an RDFa with some RDF. store.fetcher.load() creates an rdf graph When trying to load container representation with store.fetcher.load() the graph exists but is not a container the container representation.

bourgeoa commented 3 years ago

solved with https://github.com/solid/solid-panes/pull/312