-
Notifications
You must be signed in to change notification settings - Fork 245
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
chrome-storage-sync adapter (work in progress) #150
base: master
Are you sure you want to change the base?
Conversation
ack! I want to merge but many of these commits are complete rewrites (reformats?) of all the files---- can you cherry pick to just the files for this feature? |
sorry... I'm a relative git newbie. the only file that matters is chrome-storage-sync.js im not sure how to cherry pick just that one. the rest of the commits are from when i was trying to figure out how to test it. |
No worries, my git-fu is super google dependent too. Found this trick: http://jasonrudolph.com/blog/2009/02/25/git-tip-how-to-merge-specific-files-from-another-branch/ which should work nicely. |
nice find. i just wrote and posted a stackoverflow question as this update showed up on my end. |
one other issue is in using multiple lawnchair instances. chrome.storage does not provide a 'table/collection' level of abstraction. a work around could be to use an indexing system, but given how little space is provided by chrome.storage.sync, i'm not sure if this is the best way. |
I wrote a lawnchair adapter for chrome.storage.sync (only for chrome apps and extensions at the moment).
Advantages:
Disadvantages:
see https://developer.chrome.com/apps/storage.html#sync-properties
for details