-
Notifications
You must be signed in to change notification settings - Fork 32
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
Promise interface for API #30
Comments
I had started to migrate the whole library to promises and drop support for callback but I couldn't get to finish that. I'll try to continue working on that. |
@leebenson I may be late for this, but if you want some sort of promise to work you can do a workaround like this.
|
@umayr great work you have here, was wondering if you have finished promisifying the project or not ? |
@englayth try passing in a noop fn as the first argument and then the options object normally. |
recurly.api
doesn't fit the Promise interface pattern of other calls -callback
is currently required. Would be great to get this working per the other functions.The text was updated successfully, but these errors were encountered: