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

Add Support for Pages Analytics #30

Merged
merged 2 commits into from
Aug 9, 2022
Merged

Add Support for Pages Analytics #30

merged 2 commits into from
Aug 9, 2022

Conversation

tmeyer2115
Copy link
Contributor

Support was added for Pages Analytics Events and Conversion Tracking.

tatimblin and others added 2 commits July 28, 2022 16:45
Also updates the main AnalyticsConfig, AnalyticsReporter, etc. to be aliases for SearchAnayticsConfig, SearchAnalytics Reporter, etc. in order to differentiate between the Pages & Search focused parts of the library.

Additionally, introduces console debugging support enabled when the `debug` option is set.

T=unit & manual

Unit tests added for the new PagesAnalyticsReporter. Manually testing done by updating the test-site to add the new provider, fire a page view, and add a button to test click events.

Co-authored-by: Ben McGinnis <bmcginnis@yext.com>
Co-authored-by: github-actions <41898282+github-actions[bot]@users.noreply.github.com>
Adds Conversion Tracking to the Analytics Library, including:

* New standalone ConversionTrackingReporter
* Additions to PagesAnalyticsReporter to allow it to track conversions using the ConversionTrackingReporter as well as properly credit Listings and Pages for conversions that happen later
* Adds a CookieManager utility class that can be used by implementers to retrieve the Yext cookieId and set it if need be.

TEST=unit & manual

* Added unit tests for ConversionTrackingReporter and updated them for PagesAnalyticsReporter.
* Additionally updated test site to add relevant buttons for conversion tracking. To test Cookies you will need to serve with a TLS cert on your local.
@benmcginnis
Copy link
Member

LGTM, it has both commits that were already reviewed in #29 and #28

@tmeyer2115 tmeyer2115 merged commit 2305c72 into main Aug 9, 2022
@benmcginnis benmcginnis deleted the pages/trunk branch September 28, 2022 14:44
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.

None yet

3 participants