Add support for promises about debounce HOT 8 CLOSED

I can open a PR for this, as soon as the maintainer is fine making it return a Promise only when it's available on the runtime environment. In other words, if you are using oldIE then you don't get a Promise as a result if you don't load a polyfill. That also looks backwards compatible.

from debounce.

It might be worth it as a separate package at that point, tbh.

from debounce.

The first use of the debounced function which should return a promise was returning undefined

import debounce from 'debounce'

const search = (search) => {
  return new Promise((resolve,reject)=>{
   // ...
    resolve(res.data)
  })
}

const searchDebounce = debounce(search, 250)

searchDebounce("foobar").then() // threw error, promise wasn't returned, undefined was returned

// sometime later

searchDebounce("foobar").then() // working as intended

I was able to work around the issue by switching out debounce for debouce-promise.

https://github.com/bjoerge/debounce-promise

from debounce.

That's the difference between debounce and throttle.

from debounce.

So this issue seems to be alive.
Back then I ended up publishing debounce-with-result.

from debounce.

This seems like a 100% reasonable thing to expect from this library. I'll make sure I have permissions to do so tomorrow and try to get a working version published to npm.

EDIT - PR welcome. Will merge and publish, but the change is more involved than I need to be in this library right now (only incidentally stumbled upon this issue :/)

from debounce.

I mean, if https://github.com/bjoerge/debounce-promise does what we want, we don't need to over-complicate this library and just add a mention in the README.md. Unless the maintainers really care about this feature!

from debounce.

I don't think promise supports needs to live in this package. There's debounce-promise as mentioned above and also my p-debounce package.

from debounce.