-
Notifications
You must be signed in to change notification settings - Fork 3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is great! I was able to create an identity with this.
Perfect! Let's add the operations too.
|
Addition:
|
Researched a little bit on callbackify and fount Node.js implementation https://nodejs.org/dist/latest-v8.x/docs/api/util.html#util_util_callbackify_original. I swapped the parameter order to make it more developer familiar. Now it is used operation(...args, (err, res) => {
if (err) return handleError(err)
return handleSuccess(res)
}) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great!
Removes old implementation, I created a simpler one now. It includes: