Skip to content

Conversation

@fnhipster
Copy link
Collaborator

@fnhipster fnhipster commented May 21, 2025

🛠 Changes

  • Renamed the ac-store query parameter to cache-buster for clarity.
  • Introduced a short hash generator based on the configuration header content:
    • Ensures uniqueness per config change
    • Keeps URL lengths short to avoid browser GET limits. Example: https://www.aemshop.net/cs-graphql?cb=3cy4z&query=...
  • Fixed tools/pdp-metadata, which was broken after the last update to the storefront config.

TODO

Test URLs:

@aem-code-sync
Copy link

aem-code-sync bot commented May 21, 2025

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

@aem-code-sync
Copy link

aem-code-sync bot commented May 21, 2025

Page Scores Audits Google
📱 / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ / PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

Copy link
Collaborator

@sirugh sirugh left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

a few suggestions but otherwise good

… 5 and modify cache-buster parameter to 'cb' in commerce endpoints.
@devpatil7 devpatil7 self-requested a review May 22, 2025 21:09
@devpatil7 devpatil7 merged commit 7c9d223 into main May 22, 2025
4 checks passed
@devpatil7 devpatil7 deleted the cache-busting branch May 22, 2025 21:09
sirugh pushed a commit that referenced this pull request Jun 4, 2025
chore: fix 404 script tag
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.

5 participants