active_merchant icon indicating copy to clipboard operation
active_merchant copied to clipboard

Question: Is accept.js supported?

Open herberzt opened this issue 5 years ago • 7 comments

I have been using active merchant to interact with Authorize.net prior to accept.js was released. Are you all supporting that yet? I managed to find a closed issue that referenced supporting it but when searching the repo, I can't seem to find any code referencing the support.

I guess the main question should really be, does the authroize.net part support using the payment nonce instead of passing the card data in?

herberzt avatar Apr 03 '19 16:04 herberzt

Bump

herberzt avatar Apr 10 '19 16:04 herberzt

We're using activemerchant for accept.js payments, but we had to add a small monkey patch to do so.

I'd be happy to put up a PR for this, but based on the response to a different PR I have and the number of outstanding PR's, I'm not sure I'll get much of a response from those with write access to this repo.

If somebody with write access is willing to commit to reviewing and helping to get that PR merged, just let me know.

dingels35 avatar May 10 '19 15:05 dingels35

@herberzt - You willing to do some code review and QA if I put up a PR?

@curiousepic and @nfarve - You two looks like the last two people to commit changes to authorize.net in this project. Are either of you willing to help with review and merging? (One thing specifically that is needed is adding a public client key to the authnet fixture for that account, so that I can generate an accept.js nonce for remote specs.)

FYI: accept.js documentation: https://developer.authorize.net/api/reference/features/acceptjs.html

dingels35 avatar May 13 '19 19:05 dingels35

@dingels35 - thank you for reaching out! We're currently looking at how we can improve our workflows to better process incoming issues and PRs. Your contribution is most welcome and we would be happy to review it. Looking briefly at the linked documentation, it appears you can submit the public key directly via an API call using the test credentials included in the fixtures.yml file.

bayprogrammer avatar May 15 '19 15:05 bayprogrammer

@bayprogrammer - Thanks for the note and the suggestion to pull the public key from the API. It worked out well.

I have a PR up for this now - #3224. I'm looking forward to a review.

dingels35 avatar May 17 '19 12:05 dingels35

@dingels35 Thanks for your PR! I am going to ask to place it into our review pipeline and expect myself or another engineer to be able to review it soon. Great news on getting the public key via an API call!

bayprogrammer avatar May 17 '19 13:05 bayprogrammer

Marking this "of interest" before a cleanup of stale issues

curiousepic avatar Oct 26 '23 18:10 curiousepic