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 Jun 4, 2019. It is now read-only.
So my suggestion is that we use the second/third style (since Promises in the style I mocked in the first don't exist yet lol) and our Conductors expect to call success/fail appropriately with an Error object that we expect/understand. @parndt thoughts here? I have used callbacks before in controllers in a rails app and was happy (just feels objecty to not be saying "do something, then tell me how you did on it, then I'll do other stuff in response" and instead say "do this thing, and here's how you handle these optional cases".
One benefit there is that the Conductor could optionally do NEITHER of those things, at his prerogative. So you "Tell, Don't Ask" and it's up to the Conductor to do what he wishes. He's in charge. But you can give him good ideas.
The text was updated successfully, but these errors were encountered:
Sign up for freeto subscribe to this conversation on GitHub.
Already have an account?
Sign in.
https://gist.github.com/knewter/5201992
So my suggestion is that we use the second/third style (since Promises in the style I mocked in the first don't exist yet lol) and our Conductors expect to call success/fail appropriately with an Error object that we expect/understand. @parndt thoughts here? I have used callbacks before in controllers in a rails app and was happy (just feels objecty to not be saying "do something, then tell me how you did on it, then I'll do other stuff in response" and instead say "do this thing, and here's how you handle these optional cases".
One benefit there is that the Conductor could optionally do NEITHER of those things, at his prerogative. So you "Tell, Don't Ask" and it's up to the Conductor to do what he wishes. He's in charge. But you can give him good ideas.
The text was updated successfully, but these errors were encountered: