-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Chore/FAVE-37047 Switch to New Domain Names Due to The Alipay+ Network Upgrade #4
Conversation
OpenAPI Documentation
…original-repo Add Integration for Create QR Code, Update QR Code, Overseas Query, Spot Refund
fix failed timezone test
f7f8ee1
to
06b75b0
Compare
…reset-to-upstream
2fc81a1
to
81d30b1
Compare
81d30b1
to
b811232
Compare
lib/alipay/service.rb
Outdated
@@ -1,6 +1,6 @@ | |||
module Alipay | |||
module Service | |||
GATEWAY_URL = 'https://mapi.alipay.com/gateway.do' | |||
GATEWAY_URL = 'https://api-sea-global.alipayplus.com/gateway.do' |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps we can try to declare an attr_accessor
in the alipay.rb
instead of hardcoding the URL, so we can accessed globally with something like this Alipay.legacy_gateway_url
, it's easier to configure later with different value without the need to change in the gem code (eg: configure in rails initializer)
Line 25 in 63959f8
attr_accessor :pid, :key, :sign_type, :debug_mode |
679b76a
to
d15850e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 🚀 , thanks for the PR
e2fae72
to
d15850e
Compare
Description
*Refer to chloerei/alipay@master...kfit-dev:alipay:chore/FAVE-37047-reset-to-upstream to see the actual differences compared to the upstream.
Context or Ticket
https://kfit-asia.atlassian.net/browse/FAVE-37047