-
-
Notifications
You must be signed in to change notification settings - Fork 532
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
Store settings via cloud storage to easily sync across devices #1972
Comments
@ImprovedTube I'd like to try solving this issue |
wonderful! @josephShokry |
@ImprovedTube is there a cloud serive you prefer me to use or just use google drive |
hi! @josephShokry For Browser extensions (like uBlock origin) we can use chrome.storage.sync, since it doesnt require permission. It is limited to 100kb (enough yet) (and should be fastest/instant): #1656 ( chrome documentation, requires no change firefox ) Google drive also makes sense if most users are logged in to YouTube (besides it will have to request a google drive permission. "PocketTube" for example also added this feature. (We may also use Google Drive te day we continue this amazing extension: Smart History Manager (try it?) +it could consider data at myactivity.google in future) |
@ImprovedTube I am currently working on the issue and I am making progress! |
amazing! @josephShokry ( congratulations! @sfrassanito 😃 ) ( Could also sync changes all them time as an option or by default🤔.
Since we cant use the same browser user account for both(, can we?) i just meant: https://stackoverflow.com/a/48030606 (+ [safari does it automated by now] )(https://developer.apple.com/documentation/safariservices/safari_web_extensions/syncing_safari_web_extensions_across_devices_and_platforms) ) looking forward to get google drive too 😃 |
BTW, just another story about Settings data: We should count the popularity of each feature to prioritize our work. We (and other extensions) mostly have toggles & dropdowns (Next to our custom colors and video ID history / analyzer) (so that for example the minimum requirement to store 120 toggles in fixed order would be 120 bit or only 20 chars when converted to base64. So one could even allow to export/import/share profiles as a short code or URL like YouTube.com/0001000000010000010000001000) |
Than you @josephShokry and @ImprovedTube. I like the idea of having Google Drive as storage (maybe as second option, as it likely requires users to do additional authentication), as that would allow to use potentially any browser. Also, a future enhancement could be that settings can be set to automatically be pulled if the local version is older than the remote one, so the extension would automatically update its settings in all browsers. I haven't seen this in any extension yet, but I don't see any downside so far. |
Thank you @sfrassanito for this interesting issue and @ImprovedTube for the second opportunity to contribute to such a wonderful extension. in the case of the future enhancement you @sfrassanito suggested I was going to implement it but recognized that the issue is requiring not to be automatically updated instead the user should push and pull so that is what I did but I totally agree that automatically updated is better and we can make the feature to be activated or deactivated why not? |
@josephShokry, I created a new feature request for the future as you suggested. Ref. #1999. |
@ImprovedTube last thing in this issue, there is a repeated code because of the old export and import functions and the new sync are almost the same the difference is the way to save them so it would be better if the took the functionality of saving and loading settings away in another function and called this functions in the old and new methods |
⚬ PROBLEM:
Today one needs to export/import settings in JSON to keep settings synched across multiple devices.
⚬ SOLUTION:
Sync settings in cloud storage. Similar to how uBlock Origin does (Push/Pull). From one device one can Push the settings and store them, from another device one can Pull them and apply them to the local instance.
⚬ ALTERNATIVES:
⚬ RELEVANCE / SCOPE:
⚬ "SIDE EFFECTS":
⚬ CONTEXT:
//
Thank you!
The text was updated successfully, but these errors were encountered: