You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
{{ message }}
This repository has been archived by the owner on Nov 27, 2023. It is now read-only.
As discussed in #80, authentication errors are async so they can't be handled properly at the moment.
A simple fix would be to modify callbacks to be callback(error, data) instead of the current callback(data).
We could be backwards compatible by doing callback(data, error), although this would go against standard library convention, so open to opinions on this as well.
Another option would be to refactor into using promises. Or even do both (optional callbacks, and promise returns): #86 offers a nice proposal for this by @evanshortiss I'm leaning towards this and will do some work for it in the following days unless anyone has a better idea
Open for comments! These changes will be released as v2 of Robinhood-node
The text was updated successfully, but these errors were encountered:
As discussed in #80, authentication errors are async so they can't be handled properly at the moment.
A simple fix would be to modify callbacks to be
callback(error, data)
instead of the currentcallback(data)
.We could be backwards compatible by doing
callback(data, error)
, although this would go against standard library convention, so open to opinions on this as well.Another option would be to refactor into using promises. Or even do both (optional callbacks, and promise returns): #86 offers a nice proposal for this by @evanshortiss I'm leaning towards this and will do some work for it in the following days unless anyone has a better idea
Open for comments! These changes will be released as v2 of Robinhood-node
The text was updated successfully, but these errors were encountered: