urfuwo / hlx-test

Apache License 2.0
0 stars 3 forks source link

DRAFT: 507 consentless analytics page views in aem eds, w/out launch container #534

Open pahupe opened 2 months ago

pahupe commented 2 months ago

Context: Setup requested by Markus Riedinger:

Initial implementation validated with Sankalp Singh.

Fix #507

Test URLs: Before: https://main--hlx-test--urfuwo.hlx.live/blog/2023/11/idc-marketscape-sap-a-leader-cloud-enabled-sourcing-applications After: https://507-analytics-in-aem-eds--hlx-test--urfuwo.hlx.live/blog/2023/11/idc-marketscape-sap-a-leader-cloud-enabled-sourcing-applications

Validation: Open page (no ?tr parameter needed), open browser console and filter for requests containing "b/ss" in the URL. Per page we should see one request sending the (converted) Data Layer data to Analytics.

The format of the request follows the API specs:

aem-code-sync[bot] commented 2 months ago

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.

Commits * [78405fa](https://github.com/urfuwo/hlx-test/commit/78405fa9937aaf91dffdd9f826e4b49e01b3c146) :white_check_mark: (latest) * [c8a302e](https://github.com/urfuwo/hlx-test/commit/c8a302e301b2eae0e269f9425532e8e8428db8db) :white_check_mark: * [36b0e47](https://github.com/urfuwo/hlx-test/commit/36b0e47c0016a5fc90ff3596b9f6c0da74934c16) :white_check_mark: * [bf67254](https://github.com/urfuwo/hlx-test/commit/bf67254afa6b4de7a4c8368dfc05784bc88f55ba) :white_check_mark: * [fbf89fc](https://github.com/urfuwo/hlx-test/commit/fbf89fc8007af182158cb418d3c8afdbe1066edb) :white_check_mark: * [84cfb60](https://github.com/urfuwo/hlx-test/commit/84cfb60b42613722656691ada898c22ef201dfbe) :white_check_mark:
aem-code-sync[bot] commented 2 months ago
Page Scores Audits Google
/blog/2023/11/idc-marketscape-sap-a-leader-cloud-enabled-sourcing-applications PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
pahupe commented 2 months ago

Converted to DRAFT as @urfuwo mentioned that data should also be tracked in CDI