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

Move storing logic to background script #103

Open
sharon-wang opened this issue Dec 31, 2020 · 0 comments
Open

Move storing logic to background script #103

sharon-wang opened this issue Dec 31, 2020 · 0 comments
Labels
data Akita extension data storage, retrieval, organization

Comments

@sharon-wang
Copy link
Member

Currently, all of our storing logic exists in storage.js and is executed by the content script being run on the active tab. However, this may pose some issues since all tabs write to the same browser local storage and the stores occur asynchronously. The asynchronous stores may give rise to incorrect data, as data may get overwritten.

If we move our storing logic to our background script, then we can coordinate saving to storage across all tabs and avoid the problem of multiple tabs asynchronously writing to the same storage.

@sharon-wang sharon-wang added the data Akita extension data storage, retrieval, organization label Apr 4, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data Akita extension data storage, retrieval, organization
Projects
None yet
Development

No branches or pull requests

1 participant