Skip to content

Option to supress gift related events to dataLayer when on eCard pages#385

Merged
fernanDOTdo merged 8 commits into
mainfrom
ecard-supress-datalayer
Mar 19, 2026
Merged

Option to supress gift related events to dataLayer when on eCard pages#385
fernanDOTdo merged 8 commits into
mainfrom
ecard-supress-datalayer

Conversation

@StoneMoney

@StoneMoney StoneMoney commented Mar 9, 2026

Copy link
Copy Markdown
Member

@bryancasler has conducted preliminary QA already

  • For task: https://app.productive.io/2650-4site-interactive-studios-inc/tasks/16619272
  • Adds new option in configuration SupressPurchaseEcard (default false, will not change any existing installations)
  • When page type is ECARD and this SupressPurchaseEcard is true, DataLayer module will not report the EN_SUCCESSFUL_DONATION event or record payment-related pageJson values to EN_PAGEJSON_XXXX

Testing Process

  1. Use GTM tag assistant for ats.org, pointed to this page: https://secure.ats.org/page/93557/donate/1
  2. Complete the donation form, tick the box that the gift is in honor, and fill out the ecard data, and submit the form
  1. Check the tag assistant that the ECard Page does not have a EN_SUCCESSFUL_DONATION event
  2. Check the tag assistant that the ECard Page pageJsonVariablesReady's event does not contain any gift process variables

Gift process variables:
* amount
* currency
* donationLogId
* feeCover
* giftProcess
* paymentType
* receiptNumber
* recurring
* transactionId
* transactionType

  1. Test functionality again but with SuppressPurchaseEcard set to false; events should not be suppressed.

@StoneMoney
StoneMoney marked this pull request as ready for review March 11, 2026 19:45
@fernanDOTdo
fernanDOTdo merged commit 828ff7b into main Mar 19, 2026
1 check passed
@fernanDOTdo
fernanDOTdo deleted the ecard-supress-datalayer branch March 19, 2026 21:36
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