Benjamin Pollack
Benjamin Pollack
@bdewater I rebased this and made tests pass. Happy to merge if you'd like.
@slogsdon Would you mind providing a link to the documentation that supports this change?
Ah, gotcha. In that case, I can just go ahead and merge.
@nfarve This one also seems clean to merge; look okay to you?
@bdewater Is this something you're still interested in landing?
@nfarve Because you're in authorize.net, hit me you'd be a really good person to tag. This seems like it might be in line with the general work you're doing right...
Thanks for the PR. If this is still relevant to you (sorry it's been awhile!), I'd appreciate at least some information about what this is correcting/fixing and why, and ideally...
@jasonwebster Is this something you're still interested in landing?
Awesome, I'll look @albaer . I've been going through and cleaning up/landing a lot of PRs, but it's a slow burn. I'll try to get you feedback within the next...
@albaer What would you need to update this and help land it?