active_merchant
active_merchant copied to clipboard
SecurePayAU: Send order ID for payments with stored card
SecurePayAU: Send order ID for payments with stored cards
Order IDs were being sent for normal payments, but now they are also sent for payments triggered on stored cards as transactionReference
.
As per XML spec (https://auspost.com.au/payments/docs/securepay/resources/API_Card_Storage_and_Scheduled_Payments.pdf )
Also:
- Fixed broken remote tests
- Send content-type headers for best practice
Unit test results: test/unit/gateways/secure_pay_au_test.rb 24 tests, 106 assertions, 0 failures, 0 errors, 0 pendings, 0 omissions, 0 notifications 100% passed
Remote test results: test/remote/gateways/remote_secure_pay_au_test.rb 18 tests, 57 assertions, 0 failures, 0 errors, 0 pendings, 2 omissions, 0 notifications 100% passed
It would be great if someone could take a look at this. We have been using this code in production for a long time.
@rachelkirk and @curiousepic, apologies for the ping, but I'm not sure what the process is for reviewing PRs and I see that you've recently approved a PR.
We've been using this code since 2018, and submitted the PR in 2021 as per the contributing guide. We've also submitted another PR here.
Would you or somebody be able to take a look? We'd love to be able to use and further support the official gem, rather than our own fork.
PS I actually have some spec improvements to offer too, but as they depend on this PR I never bothered submitting them..
To provide a cleaner slate for the maintenance of the library, this PR/Issue is being labeled stale after 60 days without activity. It will be closed in 14 days unless you comment with an update regarding its applicability to the current build. Thank you!
Hi @aenand I'm pinging you as you have recently reviewed some PRs. This is due to be closed, however I believe it has value and is ready to go (including specs as per the guide). Can you please take a look or assign to the relevant person?
We've been using this code since 2018, and submitted the PR in 2021 as per the contributing guide. We've also submitted another PR here.
PS let me know if you'd like me to rebase it.
Hi @aenand I'm pinging you as you have recently reviewed some PRs. This is due to be closed, however I believe it has value and is ready to go (including specs as per the guide). Can you please take a look or assign to the relevant person?
We've been using this code since 2018, and submitted the PR in 2021 as per the contributing guide. We've also submitted another PR here.
PS let me know if you'd like me to rebase it.
Hey @dacook I'm so sorry this has slipped through the cracks. If you rebase this and add a changelog I will look to merge this
Thanks. It looks like the PR name should be enough for the changelog, and in fact the wiki states:
Does not update the CHANGELOG
Should I add the PR name to the CHANGELOG file in this PR, and if so can you please update the wiki to make that clearer?
I'm unable to install gems for the new ruby version 3.1:
Resolving dependencies...
nokogiri-1.11.7-x86_64-darwin requires ruby version < 3.1.dev, >= 2.5, which is incompatible with the current version,
ruby 3.1.4p223
I tried using Ruby 2.7.8, but it was also refused:
Resolving dependencies...
Bundler could not find compatible versions for gem "ruby":
In Gemfile:
ruby
activemerchant was resolved to 1.136.0, which depends on
ruby (>= 3.1)
Could not find gem 'ruby (>= 3.1)', which is required by gem 'activemerchant', in any of the relevant sources:
the local ruby installation
Therefore I can't re-run the tests until that is fixed.
@aenand I have rebased and added a changelog line as requested. Can you please review my above comments also?