nextcloud-libraries / nextcloud-browser-storage Goto Github PK

To store global attributes to be shared between components (like localStorage) but that should not survive page refresh like avatar states.

Could add another option ".pageLocal()and stores it inwindow._pageLocal` or something.

I'd use this for nextcloud/spreed#4284 because when people add an avatar when they didn't have one, there is no easy way for a user to force-refresh the avatars apart from logging out

E.g. guest name was chosen to be localStorage.getItem('nick') in the past and we decided it should e.g. persist over apps, so the same value should be used in Talk, Deck, Text, etc.
Since all apps use a scoped browser storage, we need a fixed scope for shared data like this, so all apps can use the same scope.

Proposal would be server or shared and in there we use nick or guest-name?

After we agreed on this, it should be documented in the readme.

cc @juliushaertl @ChristophWurst

PS: would a getUnscopedItem function make sense, so you don't need 2 stores, one for the guest name and one for your apps data?

Hello, please update the package on npmjs because it says unsupported engine when installing with npm.

clearOnLogout doesn't affect storage item's key of value. After the page reload it is unknown what item was set with clearOnLogout=true.

Proposal:

• add clearOnLogout flag to the scope name.
• implement new clear method

I tried to infer usage from the source code but it isn't as straightforward.

Would be good to add some usage examples in the README for volatile and persistent, and maybe best practices.

Hi 👋

I created this repo for nextcloud/server#15943 (comment). I went for the broader term browser storage, so this can cover not only the local storage APIs but potentially offer a wider API.

@sualko what are the basic operation you need for jsxc?