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

Added advantage description #16

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

Added advantage description #16

wants to merge 23 commits into from

Conversation

Edzelopez
Copy link
Member

No description provided.

Copy link
Member

@seamuslee001 seamuslee001 left a comment

Choose a reason for hiding this comment

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

Approved as per comments on CRM-374

@@ -618,7 +610,7 @@ function cdntaxreceipts_configure_inkind_fields() {
$group = array(
'title' => 'In-kind donation fields',
'extends' => array( 'Contribution' ),
'extends_entity_column_value' => array( $contrib_type_id ),
'extends_entity_column_value' => [$financialTypeID],
Copy link
Member

Choose a reason for hiding this comment

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

What fields are supposed to go into this group, and why is that not part of this PR? On test2, when I create a contribution with fund of In-kind, then this field set opens and shows

DESCRIPTION OF PROPERTY * |
APPRAISED BY * |
ADDRESS OF APPRAISER * |
ORIGINAL COST

Copy link
Member

Choose a reason for hiding this comment

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

@JoeMurray the custom fields are declared just few lines below at line 626 which is a part of the old code here - https://github.com/jake-mw/CDNTaxReceipts/blob/master/cdntaxreceipts.functions.inc#L613

@seamuslee001
Copy link
Member

@monishdeb I found an issue with your delete query and have fixed it in ee5e340 I have also tried to ensure that the inkind receipting is added as well @Edzelopez A question do we need to set the GL Code for the revenue account in the upgrade?

Edzelopez pushed a commit that referenced this pull request May 15, 2024
CRM-954: Token fixes for CDN taxreceipt thank-you form
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