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

108 form campaign script #109

Merged
merged 5 commits into from
Jan 11, 2024
Merged

Conversation

jonatan-lledo-netcentric
Copy link
Collaborator

@jonatan-lledo-netcentric jonatan-lledo-netcentric commented Jan 10, 2024

Update

The editor can add meta tags and their values in the metadata block to enable the campaign script on a page that contains a form campaign using the Pardot form block

The use case in the metadata block is as follows:
Campaign id or Business Unit id, followed by the id in the right column like this: Campaign Id | 12345

To enable the script injection the 3 things need to be used:

  • A Pardot form block that contains the form URL for the campaign
  • The Campaign ID tag and its value
  • The Business Unit ID tag and its value

If any or all are missing, the script is not injected

Test Url

/drafts/jlledo/campaign/default
(the test page also includes links with the other test cases if 1 or more items are missing)

drafts/jlledo/campaign/meta-pardot-hostname
The second one also adds an optional meta called Hostname that add optionally a different Pardot URL

Fix #108

Test URLs:

Copy link

aem-code-sync bot commented Jan 10, 2024

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
In case there are problems, just click a checkbox below to rerun the respective action.

  • Re-run PSI checks
  • Re-sync branch
Commits

Copy link

aem-code-sync bot commented Jan 10, 2024

Page Scores Audits Google
/drafts/jlledo/campaign/default PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

scripts/delayed.js Outdated Show resolved Hide resolved
scripts/delayed.js Outdated Show resolved Hide resolved
@aem-code-sync aem-code-sync bot temporarily deployed to 108-campaign-id-metadata January 11, 2024 10:01 Inactive
scripts/delayed.js Outdated Show resolved Hide resolved
@aem-code-sync aem-code-sync bot temporarily deployed to 108-campaign-id-metadata January 11, 2024 10:41 Inactive
@jonatan-lledo-netcentric jonatan-lledo-netcentric merged commit bfdf760 into main Jan 11, 2024
2 checks passed
@jonatan-lledo-netcentric jonatan-lledo-netcentric deleted the 108-campaign-id-metadata branch January 11, 2024 12:57
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.

PartsASIST: Adding Campaign ID in the Metadata Block
3 participants