Document how to turn callbacks into promises using bluebird

[rayshan]

Many may find this bluebird feature useful if they're looking for a redis library that supports promises or manually turning callbacks into promises:

var Promise = require("bluebird");
Promise.promisifyAll(require("redis"));
var db = redis.createClient...
db.getAsync(key).then()
...

More details
https://github.com/petkaantonov/bluebird/blob/master/API.md#promisification

[zacharynevin]

Ditto. I almost exclusively use promises now.

[brycebaril]

I'm not a promise user myself, but if you were to write up a blog post or detailed gist on how this library works when using promises (Especially if it demonstrates more than just one Promises implementation) I would be happy to link it from the Readme!

[zacharynevin]

Sure I would be happy to do that. Also, have you considered making node_redis compatible with the Promise.promisifyAll function on Bluebird? https://github.com/petkaantonov/bluebird/blob/master/API.md#promisepromisifyallobject-target--object-options---object

Zachary Nevin

[rayshan]

I've been using bluebird with no issues (see original comment), though I've only used about 10 methods or so. What's not compatible?

[zacharynevin]

I probably just did something wrong. I got the "doesn't have a method 'then' "error.

Zachary Nevin

On Jun 21, 2014, at 3:09 PM, Ray Shan [email protected] wrote:

I've been using bluebird with no issues (see original comment), though I've only used about 10 methods or so. What's not compatible?

—
Reply to this email directly or view it on GitHub.

[rayshan]

Are you using 2.x? In 1.x you have to promisifyAll a newed instance of redis. In 2.x this one line works for me:

Promise.promisifyAll(require("redis"));

[zacharynevin]

Yeah it works for me now.

[komocode]

Do you have an example on how to use the multi as promises?

[maxbrieiev]

For multi only exec should return a promise. This lib handles it for you: https://github.com/maxbrieiev/promise-redis

See the very last paragraph.

[blainsmith]

In order to better support this project and its new group of collaborators under a new org we are trying to clean up and close issues older than 6 months. Your issue may have been fixed since the issue was open, however, if you are still experiencing a problem please re-open this issue and we will label it accordingly.

[BridgeAR]

Reference #864