Implement hoodie.reset()

[gr2m]

Reset hoodie client and emit reset events so plugins can reset as well. Should implement a way so that plugins can hook into the reset with async operations

hoodie.reset().then(function () {
  // clean slate again
})

And plugins could do something like

hoodie.on('reset', function (options) {
  options.promise = options.promise.then(function () {
    // do some async reset here
  })
})

^^ This is just a suggestion, maybe there are better patterns for it :)

• Update README as part of PR

[danreeves]

Hi 👋

I'm gunna try to make a PR for this tonight. Sound good?

[gr2m]

Sounds amazing :) All yours

[gr2m]

Hey @danreeves what’s status?

[danreeves]

Hi @gr2m, finally had some time to look at this tonight. I've had some issues getting hoodie@next and hoodie-client working together but that's maybe another issue. I've made a WIP PR here: #32

Looking like the right direction?

[gr2m]

Awesome I'll check it out tomorrow, thanks a ton!

[Guria]

@gr2m I am not guru in api design, but options.promise = options.promise.then looks ugly for me. can't we just return promise and manage it from callback caller instead?

[gr2m]

Yeah I’m wondering if there is a nicer API :) But the one with options.promise is the best shot right now, as we can’t get the value that is returned by event handlers

hoodie.on('reset', function () {
  return new Promise(...) // this goes nowhere
})

hoodie.trigger('reset')

[gr2m]

@Guria here’s an alternative API suggestion to intercept things like reset events: #42

[danreeves]

@gr2m looks similar to something I was working on to get the .reset() call to be thenable without racing the event listeners: danreeves@9ded20e

edit: because options.promise = options.promise.then didn't appear to be working as expected

[gr2m]

@danreeves clever!!

[gr2m]

@danreeves let’s put hoodie.reset on hold for now. We have to think about the hooks as we need it for the glue code between hoodie.store and hoodie.account, but hoodie.reset only becomes relevant with plugins which we put out of scope for the first release. I like your options.hooks idea, maybe we can use that implementation for the hooks in sign in / sign out

[danreeves]

Yeah, that makes sense