Conversation
There was a problem hiding this comment.
Pull request overview
This pull request addresses two critical bugs in the Summit Events application:
-
Matching metadata fix: Custom metadata contact matching mappings were incorrectly copying "matching only" fields into newly created contacts, leads, and person accounts. The fix introduces a new
removeMatchOnlyFieldsmethod that filters out these fields before record creation. -
QR code rendering fix: QR code images in transactional emails were not rendering correctly. The fix refactors the email template processing to properly handle image encoding (using the
_IM1_to_IM5_format) and improves link encoding handling.
Changes:
- Added
removeMatchOnlyFieldsmethod to filter out matching-only fields before creating new Contact/Lead/Account records - Updated SOQL query to include
Matching_Only__cfield in mapping retrieval - Refactored email template processing to handle image encoding for QR codes
- Added comprehensive unit tests for the new
removeMatchOnlyFieldsfunctionality - Fixed typo in method name from
savTasktosaveTask
Reviewed changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| force-app/main/default/classes/SummitEventsContactMatching.cls | Implements removeMatchOnlyFields method and getFieldAPINameForObjectType helper; updates SOQL to include Matching_Only__c field |
| force-app/test/default/classes/SummitEventsContactMatching_TEST.cls | Adds comprehensive unit tests for removeMatchOnlyFields covering Contact, Lead, Account, and edge cases |
| force-app/main/default/classes/SummitEventsRegistration.cls | Refactors email template processing to handle image encoding (_IM1_ format) and improves link encoding logic; fixes savTask typo |
| force-app/test/default/classes/SummitEventsRegistration_TEST.cls | Updates test to include QR code field reference in email content |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Critical Changes
Changes
Issues Closed