If sync is not enabled, we revert back to basic chrome.storage.local
instead of sync. Yes they can be.


On Tue, Mar 11, 2014 at 6:37 AM, Rick Harding <notifications@github.com>wrote:

> In bookie/static/js/bookie/model.js:
>
> > +                });
> > +            } else {
> > +                chrome.storage.local.set({
> > +                    'api_url': this.get('api_url'),
> > +                    'api_username': this.get('api_username'),
> > +                    'api_key': this.get('api_key'),
> > +                    'cache_content': this.get('cache_content')
> > +                });
> > +            }
> > +
> > +            chrome.storage.local.set({
> > +                'last_bmark': this.get('last_bmark'),
> > +                'sync_config': sync
> > +            });
> > +
> > +            // localStorage.setItem('api_url', this.get('api_url'));
>
> so should we set localStorage if sync is not enabled? Or can these
> commented out lines be removed?
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/bookieio/Bookie/pull/345/files#r10458271>
> .
>