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

CRM-21577 add links to be able to add payments from the contribution … #11432

Merged
merged 1 commit into from
Dec 19, 2017

Conversation

eileenmcnaughton
Copy link
Contributor

…block

Overview

Adds links to the payment block allowing a payment to be made (if applicable). This opens the option for us to use the block in more places where we have many to one payments & adds some convenience for users

Before

No links
screenshot 2017-12-19 14 39 15

After

screenshot 2017-12-19 14 40 04

Technical Details

This is a spin off from #10776 just adding the links.

@eileenmcnaughton
Copy link
Contributor Author

@Monish can you review this - we can consider #10776 after.

@monishdeb
Copy link
Member

monishdeb commented Dec 19, 2017

As we are already providing those two payment (action) links against a contribution, I don't feel that there is need to show them again below its payment. Also this could be otherwise implemented via extension using crmRegion = payment-info-block . So in my opinion I don't think it should be core change ? Or am I missing something here in terms of usability ?

@eileenmcnaughton
Copy link
Contributor Author

eileenmcnaughton commented Dec 19, 2017

@monishdeb once we have those links in the payment block we can put it on the contribution form / membership form etc in place of the one-2-one fields.

i think it's a mild advantage in usablility (since in the cases I looked at the links were not easily available, ie in the screenshots you get 'edit' but not 'add payment') but in terms of being able to address the problems with us misprocessing payments due to the forms showing it as a one to one rather than a many to one it's an important piece.

@monishdeb
Copy link
Member

Oh ok, I agree. So I did a thorough testing on my local and its all working fine. This is a sub-PR which pulls most of the code from #10776 (which I already tested earlier) except the changes made for backoffice contribution.

Merging now :)

@monishdeb monishdeb merged commit 40a86fc into civicrm:master Dec 19, 2017
@monishdeb monishdeb deleted the payment_links branch December 19, 2017 15:16
sluc23 pushed a commit to ixiam/civicrm-core that referenced this pull request Jan 10, 2018
CRM-21577 add links to be able to add payments from the contribution …
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.

2 participants