Skip to content

Issue 69#84

Merged
varunmitra merged 4 commits intomainfrom
issue-69
Mar 13, 2026
Merged

Issue 69#84
varunmitra merged 4 commits intomainfrom
issue-69

Conversation

@varunmitra
Copy link
Copy Markdown
Collaborator

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 #69

Test URLs:

git-subtree-dir: plugins/experimentation
git-subtree-split: 0356a223f70ad6f6a4291d50757221926d12872e
Enable experimentation support: update ESLint ignores to include plugins/, add a sidekick plugin config (tools/sidekick/config.json) and a small sidekick script (scripts/sidekick.js) to toggle the external DA experiment module. Add getAllMetadata helper (scripts/shared.js) to collect scoped metadata, introduce AUDIENCES and an experimentation context in scripts/scripts.js, and dynamically load the plugin's eager/lazy entry points when experimentation metadata or sidekick is present. Also wire sidekick import/sidekick-ready handling and adjust DA preview/experiment import logic.
Replace textContent with innerHTML for various UI elements in plugins/experimentation/src/preview.js so labels and action link labels can contain HTML markup. Affected functions: createButton, createPopupItem, createPopupDialog, and createToggleButton. Also minor whitespace cleanups around link/button creation.
@aem-code-sync
Copy link
Copy Markdown

aem-code-sync bot commented Mar 13, 2026

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
Copy Markdown

aem-code-sync bot commented Mar 13, 2026

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

@varunmitra varunmitra marked this pull request as draft March 13, 2026 19:09
@varunmitra varunmitra marked this pull request as ready for review March 13, 2026 19:11
@varunmitra varunmitra requested a review from kunwarsaluja March 13, 2026 19:11
@varunmitra varunmitra merged commit 19cee70 into main Mar 13, 2026
2 checks passed
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.

[Phase 2] - Experimentation in DA+EDS

2 participants