Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

chore: update the experimentation plugin to the latest version #48

Merged
merged 5 commits into from
Nov 28, 2023

Conversation

ramboz
Copy link
Collaborator

@ramboz ramboz commented Nov 27, 2023

Just updating the reference to the experimentation plugin to its latest version that contains a few bug fixes.

Test url: https://update-experimentaion-plugin--wknd--hlxsites.hlx.live/

Copy link

aem-code-sync bot commented Nov 27, 2023

Hello, I'm the AEM Code Sync Bot and I will run some test suites that validate the page speed.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-run PSI Checks

Copy link

aem-code-sync bot commented Nov 27, 2023

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

Copy link
Collaborator

@iuliag iuliag left a comment

Choose a reason for hiding this comment

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

LGTM in terms of code changes.
2 things I noticed while validating that require configuration outside of this PR or better alignment:

  • adjust headers to be able to get and show the experimentation results from RUM
  • console log says experimentation will not run and pill is gray, though selected variant in window.hlx.experiment is one of the challengers
Screenshot 2023-11-27 at 12 04 21

plugins/experimentation/README.md Outdated Show resolved Hide resolved
@ramboz
Copy link
Collaborator Author

ramboz commented Nov 27, 2023

@iuliag thanks for the feedback!
Agreed, we need to work on the pill to update it to the v3 APIs, etc.

d2028dc fix: simulation should work even if audience is not resolved
f0904be fix: typo in readme

git-subtree-dir: plugins/experimentation
git-subtree-split: d2028dca464571ddbd6791099462b7708523a5bc
Copy link

aem-code-sync bot commented Nov 27, 2023

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

@ramboz ramboz merged commit c276d5b into main Nov 28, 2023
2 checks passed
@ramboz ramboz deleted the update-experimentaion-plugin branch November 28, 2023 08:04
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.

3 participants