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

BGE: custom lookup mappings #4082

Merged
merged 24 commits into from
Mar 11, 2019
Merged

BGE: custom lookup mappings #4082

merged 24 commits into from
Mar 11, 2019

Conversation

bethbrains
Copy link
Contributor

@bethbrains bethbrains commented Feb 26, 2019

--Custom lookup mappings are supported in BGE.

Critical Changes

Changes

Batch Gift Entry Enhancements

  • You can now include any Lookup fields from Opportunities & Payments in Batch Gift Entry.

Issues Closed

New Metadata

Deleted Metadata

@bethbrains
Copy link
Contributor Author

**lurch: attach W-029639

@LurchTheButler
Copy link

Tracking W-029639

};

// these fields have been whitelisted since they have special handling in BDI
Set<String> allowedFields = new Set<String>{
'DonationCampaignImported__c'
'Donation_Record_Type_Name__c'
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This pattern is no longer needed for DonationCampaignImported__c, but is still needed for Donation Record Type, which came in a separate sibling branch. Added it here so it wasn't an empty set in this branch.

@bethbrains bethbrains changed the base branch from feature/bge-campaign-lookup to master February 27, 2019 17:40
linkCol.typeAttributes = '{"label":{"fieldName":"'+fieldLabel+'"},"target":"_blank"}';
columns.add(linkCol);

if (dataImportDFR.type == Schema.DisplayType.Reference) {
Copy link
Contributor

Choose a reason for hiding this comment

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

😁

Copy link
Contributor

@ptewson-sfdo ptewson-sfdo left a comment

Choose a reason for hiding this comment

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

Approved!

Copy link
Contributor

@ptewson-sfdo ptewson-sfdo left a comment

Choose a reason for hiding this comment

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

Lookups for dayyyyzzz

Copy link
Contributor

@ptewson-sfdo ptewson-sfdo left a comment

Choose a reason for hiding this comment

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

Looks good!

@ptewson-sfdo ptewson-sfdo merged commit f778b82 into master Mar 11, 2019
@ptewson-sfdo ptewson-sfdo deleted the feature/bge-any-lookup branch March 11, 2019 22:45
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.

5 participants