Skip to content

PDP: source review rating from JSON-LD, load BV on intersection - #887

Open
davidnuescheler wants to merge 1 commit into
mainfrom
pdp-reviews-jsonld
Open

davidnuescheler wants to merge 1 commit into
mainfrom
pdp-reviews-jsonld

Conversation

@davidnuescheler

@davidnuescheler davidnuescheler commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

https://pdp-reviews-jsonld--vitamix--aemsites.aem.page/us/en_us/products/ascent-x2
https://pdp-reviews-jsonld--vitamix--aemsites.aem.page/drafts/uncled/blenders

Summary

  • PDP: display star rating/count from the product JSON-LD's aggregateRating instead of the Bazaarvoice rating_summary widget; the summary links to the BV-loaded review details section.
  • PDP: BV reviews script now loads via IntersectionObserver (when the reviews section nears the viewport) instead of a fixed 500ms timer, fixing a bug where BV wasn't loading at all.
  • plp-data.js / product-row.js / products.js: source reviewAverage/reviewCount from the product index's ratingValue/reviewCount columns instead of fetching/joining the separate reviews.json spreadsheet.
  • product-list: fixed inconsistent "Add to Comparison list" vs "View Comparison List" casing (per PR Product-list: compare icon toggle, conditional Starting At, count-sorted filters #846 review feedback), and replaced the compare button's background-image icon with a real <img> element.

🤖 Generated with GitHub Copilot

- PDP: display star rating/count from the product JSON-LD's
  aggregateRating instead of the Bazaarvoice rating_summary widget;
  the summary links to the BV-loaded review details, which now load
  via IntersectionObserver instead of a fixed timer.
- plp-data.js/product-row.js/products.js: source reviewAverage/
  reviewCount from the product index's ratingValue/reviewCount
  columns instead of fetching/joining reviews.json.
- product-list: fix inconsistent "Add to Comparison List" casing and
  swap the compare button's background-image icon for a real <img>.
@aem-code-sync

aem-code-sync Bot commented Sep 10, 2026

Copy link
Copy Markdown

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 all PSI checks
  • Re-run failed PSI checks
  • Re-sync branch
Commits

@davidnuescheler
davidnuescheler requested review from dylandepass and fkakatie and removed request for fkakatie September 10, 2026 17:46
@aem-code-sync

aem-code-sync Bot commented Sep 10, 2026

Copy link
Copy Markdown
Page Scores Audits Google
📱 /drafts/uncled/blenders PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /drafts/uncled/blenders PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
📱 /us/en_us/products/ascent-x2 PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /us/en_us/products/ascent-x2 PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

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.

2 participants