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

LocalStorage: replace by indexedDB/WebSQL(?) sync state, control version, data prefix #268

Closed
szymonlesisz opened this issue Sep 14, 2018 · 1 comment · Fixed by #306
Closed
Assignees

Comments

@szymonlesisz
Copy link
Contributor

We need to decide if we still want to use localStorage or maybe indexedDB/websql to store application data.

Make sure that LocalStorage/SessionStorage are synchronized across browser tabs (two wallets opened at once) Example: creating and saving different instances (passphrases) in two tabs

We should be also prepared for scenario where LocalStorage data schema will be changed without backward compatibility. To prevent errors there should be "version" flag which will be compared with expected and in case of incompatibility clears them or translate them into new schema

@szymonlesisz szymonlesisz self-assigned this Oct 10, 2018
@prusnak prusnak transferred this issue from trezor/trezor-wallet Jul 10, 2019
@slowbackspace
Copy link
Contributor

Basic integration of indexedDB for suite-web/desktop is already done (#306), progress of native implementation is tracked in #8

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

Successfully merging a pull request may close this issue.

2 participants