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

Add "pledge id" as column and filter in Contribution Detail report. #16868

Merged
merged 1 commit into from
Mar 31, 2020

Conversation

twomice
Copy link
Contributor

@twomice twomice commented Mar 20, 2020

Overview

The Contribution Detail report currently provides no information about any pledge to which the contribution may be linked. This small change will allow display of a Pledge ID, (if any), and likewise filtering by Pledge ID (as an integer)

Before

No pledge-related data in the report.

After

New column: "Pledge ID"
New filter: "Pledge ID"

Technical Details

Added a new table to $_columns, conditionally joining that table in from().

Comments

I acknowledge that this report has been stretched and pulled in sometimes conflicting directions, usually related to soft-credits vs contributions vs. both. However, this small improvement doesn't overlap with that conflict and seems to be a simple value add with no technical downside.

@civibot
Copy link

civibot bot commented Mar 20, 2020

(Standard links)

@civibot civibot bot added the master label Mar 20, 2020
@Stoob
Copy link
Contributor

Stoob commented Mar 20, 2020

Thanks for joining these tables Allen

@yashodha
Copy link
Contributor

@twomice looks good

@jitendrapurohit
Copy link
Contributor

I've tested this manually on a local setup and confirmed that the newly added Pledge ID works fine in displaying the value in results as well as filtering the result. +1 to get this merged.

@yashodha
Copy link
Contributor

@twomice @jitendrapurohit merging this, thanks!

@yashodha yashodha merged commit deae79e into civicrm:master Mar 31, 2020
@twomice
Copy link
Contributor Author

twomice commented Mar 31, 2020

Thank you @yashodha !

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants