Skip to content
This repository was archived by the owner on Dec 9, 2025. It is now read-only.

[CCEX-246816] Adding base Query param to test PREnvs#739

Merged
express-milo-merge-bot[bot] merged 4 commits intostagefrom
frictionless-experiment
Dec 3, 2025
Merged

[CCEX-246816] Adding base Query param to test PREnvs#739
express-milo-merge-bot[bot] merged 4 commits intostagefrom
frictionless-experiment

Conversation

@shairil
Copy link
Collaborator

@shairil shairil commented Oct 30, 2025

Summary

Adding the base query param to help test the HZ PR-env.


Jira Ticket

Resolves: MWPW-NUMBER


Test URLs

Env URL
Before https://main--express-milo--adobecom.aem.page/express/
After https://frictionless-experiment--express-milo--adobecom.aem.page/express/?martech=off

Verification Steps

  • Steps to reproduce the issue or view the new feature.
  • What to expect before and after the change.

Potential Regressions


Additional Notes

(If applicable) Add context, related PRs, or known issues here.

@aem-code-sync
Copy link

aem-code-sync bot commented Oct 30, 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

@codecov-commenter
Copy link

codecov-commenter commented Oct 30, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 69.80%. Comparing base (c977780) to head (ef022c1).
⚠️ Report is 1 commits behind head on stage.

Files with missing lines Patch % Lines
...tionless-quick-action/frictionless-quick-action.js 0.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #739      +/-   ##
==========================================
- Coverage   70.83%   69.80%   -1.03%     
==========================================
  Files         109      108       -1     
  Lines       25446    25367      -79     
==========================================
- Hits        18025    17708     -317     
- Misses       7421     7659     +238     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@shairil shairil changed the title [TEST] Adding base Query param to test PREnvs [CCEX-246816] Adding base Query param to test PREnvs Nov 11, 2025
@aem-code-sync
Copy link

aem-code-sync bot commented Nov 11, 2025

Page Scores Audits
📱 /express/?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS
🖥️ /express/?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS
📱 /express/?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS
🖥️ /express/?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS

@aem-code-sync aem-code-sync bot temporarily deployed to frictionless-experiment November 11, 2025 06:28 Inactive
@shairil shairil marked this pull request as ready for review November 11, 2025 06:29
@github-actions github-actions bot added the Ready for Review Ready for peer review. label Nov 11, 2025
@fullcolorcoder fullcolorcoder added Ready for QA and removed Ready for Review Ready for peer review. labels Nov 14, 2025
@nateyolles nateyolles added this to the Express-25.49 milestone Dec 1, 2025
@hadobe hadobe added Ready for Stage All requested changes have been made, approvals received, ready for next stage batch and removed Ready for QA labels Dec 2, 2025
@express-milo-merge-bot
Copy link
Contributor

Skipped merging 739: [CCEX-246816] Adding base Query param to test PREnvs due to failing or running checks

1 similar comment
@express-milo-merge-bot
Copy link
Contributor

Skipped merging 739: [CCEX-246816] Adding base Query param to test PREnvs due to failing or running checks

@express-milo-merge-bot express-milo-merge-bot bot merged commit 11ff160 into stage Dec 3, 2025
10 of 11 checks passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Design Review Not Required Ready for Stage All requested changes have been made, approvals received, ready for next stage batch

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants