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: allow admin to select Advanced or Standard card processing connection when onboarding PayPal donations #6867

Merged

Conversation

ravinderk
Copy link
Collaborator

@ravinderk ravinderk commented Aug 7, 2023

Description

This pull request implements a feature to allow the admin to select an Advanced or Standard card processing connection when onboarding PayPal donations.

Affects

PayPal Donations admin onboarding user flow.

Visuals

image

Testing Instructions

Admin should able to onboard to PayPal donations without any error.

  • A modal should display to admin to select account type when click on connect button.
  • Admin should only see smart buttons if admin selects standard card processing.
  • Admin should see smart buttons and hosted card fields if admin selects advance card processing.

Pre-review Checklist

  • Acceptance criteria satisfied and marked in related issue
  • Relevant @unreleased tags included in DocBlocks
  • Includes unit tests
  • Reviewed by the designer (if follows a design)
  • Self Review of code and UX completed

@ravinderk ravinderk force-pushed the feat/paypal-donations-select-account-type-on-onboarding branch from 66b0d2e to 8d062c3 Compare August 9, 2023 05:05
@ravinderk ravinderk self-assigned this Aug 9, 2023
@ravinderk ravinderk marked this pull request as ready for review August 9, 2023 05:16
@ravinderk
Copy link
Collaborator Author

Nice progress, @ravinderk! I left a couple comments. Also, when I switched to a country that did not support PPCP and clicked "Connect with PayPal Live" it just switched the button to "Processing..." for a moment and then flipped back, never displaying the PayPal modal.

I believe this is because we haven't deployed the server. Anything holding us back from taking the server changes live?

@JasonTheAdams You can merge and deploy https://github.com/impress-org/givewp-gateway-server/pull/21.

Copy link
Contributor

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@ravinderk I'm trying to get the gateway server working locally. In the meantime, I noticed that if there's an issue fetching the Partner Link the UI just does nothing. No error message or anything. We should inform the user:

There was an issue retrieving a link to connect to PayPal. Please try again. If the issue continues please contact an administrator.

@JasonTheAdams
Copy link
Contributor

@ravinderk I also just noticed that if the connect button is visited the icon disappears.

image

@ravinderk
Copy link
Collaborator Author

@ravinderk I'm trying to get the gateway server working locally. In the meantime, I noticed that if there's an issue fetching the Partner Link the UI just does nothing. No error message or anything. We should inform the user:

There was an issue retrieving a link to connect to PayPal. Please try again. If the issue continues please contact an administrator.

Resolved a836db5

@ravinderk I also just noticed that if the connect button is visited the icon disappears.

image

Resolved 7e0e57e

@ravinderk
Copy link
Collaborator Author

@JasonTheAdams I implemented the feature as you requested in slack.

Fix: 2e7c636

image

@JasonTheAdams
Copy link
Contributor

@ravinderk Let's get rid of the "APIs Connected" text, as it's kind of meaningless.

@ravinderk
Copy link
Collaborator Author

@ravinderk Let's get rid of the "APIs Connected" text, as it's kind of meaningless.

Fixed: 086bf6f

image

@jdghinson
Copy link
Contributor

@ravinderk Let's get rid of the "APIs Connected" text, as it's kind of meaningless.

Fixed: 086bf6f

image

@ravinderk Please change the "disconnect" color from blue to red

@ravinderk
Copy link
Collaborator Author

@jdghinson

@ravinderk Please change the "disconnect" color from blue to red

fixed: 74f06d4
image

Copy link
Contributor

@JasonTheAdams JasonTheAdams left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work, @ravinderk! Let's release the gateway and then send this to QA!

Copy link
Member

@rickalday rickalday left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Passed manual QA tests

@ravinderk ravinderk merged commit a8be326 into develop Aug 29, 2023
11 of 16 checks passed
@ravinderk ravinderk deleted the feat/paypal-donations-select-account-type-on-onboarding branch August 29, 2023 04:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants