Add PDP events

[alandana]

• upgrade to PDP v0.2.3-alpha7 which has support for ACDL integration
  • product page view event
  • product context
• upgrade to dropins tools v0.25.0

Please always provide the GitHub issue(s) your PR is for, as well as test URLs where your change can be observed (before and after):

Fix #

Test URLs:

• Before: https://main--aem-boilerplate-commerce--hlxsites.hlx.live/products/crown-summit-backpack/24-MB03
• After: https://pdp-acdl--aem-boilerplate-commerce--hlxsites.hlx.live/products/crown-summit-backpack/24-MB03
  https://pdp-acdl--aem-boilerplate-commerce--hlxsites.hlx.live/products/abominable-hoodie/MH09

[aem-code-sync]

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

• a22ecea 🔄 (latest)
• e9e0533 ✅
• 6632f14 ✅
• 7801527 ✅
• 58f3d6c ✅
• 358e596 ✅
• ee835ea ✅
• fccf010 ✅
• 2c2b9a2 ✅
• eae07b7 ✅
• 9cde325 ✅
• 4e5410a ✅
• 338b311 ✅

[aem-code-sync]

Page	Scores	Audits	Google
/products/abominable-hoodie/MH09
/products/crown-summit-backpack/24-MB03

[herzog31]

According to the schema, productId should be a number, not a string. Is there a way to customize the canonicalUrl? As in the context of Edge Delivery the canonical URL is usually different than what's returned by the API.

[alandana]

According to the schema, productId should be a number, not a string. Is there a way to customize the canonicalUrl? As in the context of Edge Delivery the canonical URL is usually different than what's returned by the API.

@herzog31 What do you mean by customizing the canonicalURL? From the dropin or from EDS?
I realized I mapped the canonicalUrl to urlKey, but the API response also provides url. Would that be better?

[herzog31]

LGTM 🙌

[fnhipster]

Does this mean that ACDL is on opt-in only basis? I thought it was implemented OOTB in all dropins. Is this a pattern we should follow for all other dropins?

[alandana]

No, I don't think this should be a pattern followed in the other dropins, and eventually I might even remove it from the PDP dropin.
I had to add this because I have no control of how/when Bulk will update the PDP version and they are already handling the data collection on the product details block directly and I didn't want to interfere with that.