Skip to content

feat(pages-components): add enableYextAnalytics to window #90

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

Merged
merged 4 commits into from
Apr 7, 2025

Conversation

mkilpatrick
Copy link
Collaborator

This adds the enableYextAnalytics function to the window. According to our documentation:

The site implementer must ensure their website is explicitly opted out of Analytics; for website implemented using Pages, this can be done by setting requireOptIn to true on the AnalyticsProvider component.
After user’s relevant consent, enable the Yext analytics pixel by calling the function window.enableYextAnalytics();.

enableYextAnalytics was never something exposed by this library. It was always something the Consulting starter exposed and became standard practice. Since it's part of our official documentation the library should add the explicit support.

Copy link

changeset-bot bot commented Apr 7, 2025

⚠️ No Changeset found

Latest commit: cd0705f

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

asanehisa
asanehisa previously approved these changes Apr 7, 2025
benlife5
benlife5 previously approved these changes Apr 7, 2025
@mkilpatrick mkilpatrick dismissed stale reviews from benlife5 and asanehisa via cd0705f April 7, 2025 21:47
@mkilpatrick mkilpatrick merged commit 2b50890 into main Apr 7, 2025
12 of 14 checks passed
@mkilpatrick mkilpatrick deleted the enable-yext-analytics branch April 7, 2025 21:50
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.

3 participants