Skip to content
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

feat(connector): [Stripe] Add support for WeChat Pay and Qr code support in next action #1555

Merged
merged 20 commits into from
Jul 6, 2023

Conversation

Sangamesh26
Copy link
Contributor

Type of Change

  • New feature

Description

Add support for WeChat Pay, we chat pay in stripe doesn't follow redirection.Qr code url is given in the next action, this qr code has to be shown in sdk.

API contract for WeChat Pay:

    "payment_method": "wallet",
    "payment_method_type": "we_chat_pay",
    "payment_method_data": {
        "wallet": {
            "we_chat_pay": {}
        }
    },

Additional Changes

  • This PR modifies the API contract

Motivation and Context

Add support for WeChat Pay payment method

How did you test it?

Manual

Qr code url in next action
Screen Shot 2023-06-27 at 6 16 14 PM

Qr code
Screen Shot 2023-06-27 at 6 15 53 PM

Checklist

  • I formatted the code cargo +nightly fmt --all
  • I addressed lints thrown by cargo clippy
  • I reviewed submitted code

@Sangamesh26 Sangamesh26 added A-connector-integration Area: Connector integration A-core Area: Core flows C-feature Category: Feature request or enhancement S-waiting-on-review Status: This PR has been implemented and needs to be reviewed A-payment-methods Area: Payment Methods labels Jun 27, 2023
@Sangamesh26 Sangamesh26 self-assigned this Jun 27, 2023
@Sangamesh26 Sangamesh26 requested review from a team as code owners June 27, 2023 14:05
@SanchithHegde SanchithHegde added S-waiting-on-author Status: This PR is incomplete or needs to address review comments S-needs-conflict-resolution Status: This PR needs conflicts to be resolved by the author labels Jun 28, 2023
@Sangamesh26 Sangamesh26 added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed and removed S-waiting-on-author Status: This PR is incomplete or needs to address review comments S-needs-conflict-resolution Status: This PR needs conflicts to be resolved by the author labels Jul 5, 2023
@jagan-jaya jagan-jaya added S-waiting-on-author Status: This PR is incomplete or needs to address review comments and removed S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Jul 5, 2023
@Sangamesh26 Sangamesh26 added S-waiting-on-review Status: This PR has been implemented and needs to be reviewed and removed S-waiting-on-author Status: This PR is incomplete or needs to address review comments labels Jul 5, 2023
@srujanchikke srujanchikke added the P-high Priority: High label Jul 6, 2023
@SanchithHegde SanchithHegde added this pull request to the merge queue Jul 6, 2023
Merged via the queue into main with commit a15a77d Jul 6, 2023
@SanchithHegde SanchithHegde deleted the we_chat_pay_qr_code branch July 6, 2023 10:30
@sahkal sahkal added the M-api-contract-changes Metadata: This PR involves API contract changes label Jul 6, 2023
@SanchithHegde SanchithHegde removed P-high Priority: High S-waiting-on-review Status: This PR has been implemented and needs to be reviewed labels Jul 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-connector-integration Area: Connector integration A-core Area: Core flows A-payment-methods Area: Payment Methods C-feature Category: Feature request or enhancement M-api-contract-changes Metadata: This PR involves API contract changes
Projects
No open projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

6 participants