Mike Flynn
Mike Flynn
I'm back from vacation and catching up! `RequestEnvelope` works for me. Not sure about the breaking things out at first blush. I'd need to see it in action...and I'm headed...
Yeah, this refactor kept getting larger and then things stalled. Any help in additional changes, lists of what's needed, or testing would be appreciated!
This change makes sense and is simple enough, but you've got a couple of missing double quotes in your type JSON tags.
What's the status on this PR? This conflicts with the current master but if it's still needed, I can merge. (Also, please don't hesitate to ping me on Twitter [@thatmikeflynn]...
That would be great! Have either of you set that up with GitHub PRs before? Any way you can point me in the right direction?
So this got kind of lost in the shuffle. Still important? If someone can resolve the conflict we can merge.
Thanks for the heads up!
I've been traveling and unable to check in on this, but is this still and issue with Amazon's servers?
The Destroy method is a nice addition, but the module.exports will error if you are using the library outside of Node.
Ha! Wow, it took a while for someone to find this bug! To be clear, you're just saying you want the `(:from msg)` to be set there rather than that...