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

Backoffice contribution credit card form unable to update the billing block #331

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

monishdeb
Copy link

@monishdeb monishdeb commented Sep 18, 2020

Overview

Steps to replicate:

  1. Install and enable iATS paymentextension
  2. Add and configure an iATS Payments ACH/EFT payment type payment processor.
  3. Enable USD and CAD currencies
  4. Go to 'Submit Credit Card Contribution' backoffice form
  5. Choose payment processor created at step 2

Issue:

  1. Payment block renders payment block without check template
  2. Switching currency doesn't update the payment block.

Before

crm-598-before

After

crm-598-after

ping @adixon @eileenmcnaughton @KarinG

@adixon
Copy link
Contributor

adixon commented Sep 18, 2020

Ah, very nice, thanks for this. Might even be in time for our next release that's due out shortly.

Especially like the animated gifs!

@monishdeb
Copy link
Author

Thanks @adixon I have just now pushed additional fixes and updated the AFTER screencast, to provide a glimpse of how the CC backoffice form is behaving now.

@KarinG
Copy link
Contributor

KarinG commented Sep 18, 2020

Great @monishdeb 👍

@adixon - note there is a companion PR for Core (by Monish) that will need to be merged to make this work.

@KarinG
Copy link
Contributor

KarinG commented Sep 18, 2020

@adixon - this is the core PR -> civicrm/civicrm-core#18517

@monishdeb
Copy link
Author

@adixon @KarinG I think this PR can be merged as earlier it was depending on core PR civicrm/civicrm-core#18517 which got merged in Civi 5.35.

@KarinG
Copy link
Contributor

KarinG commented Sep 23, 2021

Ah right - it was the issue where I suggested that the "SUBMIT CREDIT CARD CONTRIBUTION" button really should be relabeled if it would now also allow people to submit "ACH/EFT" but I was overruled 🙂

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.

None yet

3 participants