-
Notifications
You must be signed in to change notification settings - Fork 163
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
[Release] Stage to Main #2742
[Release] Stage to Main #2742
Conversation
…desktop (#2715) * Update utils.js * Update utils.js * Update utils.js * Update utils.js * Update utils.js * Update utils.js * Update body.html * Update utils.test.js
…r feature (#2736) * mepgeofolder * Update libs/features/personalization/preview.js Co-authored-by: Mark Perry <[email protected]> * MWPW-156545 [MILO][MEP] Info in MEP button panel is incorrect (#2729) * stash * add override existance to if check --------- Co-authored-by: Mark Perry <[email protected]>
Testing can start @adobecom/miq-sot @adobecom/bacom-sot @adobecom/homepage-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot |
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2742 +/- ##
==========================================
- Coverage 95.89% 95.88% -0.02%
==========================================
Files 173 173
Lines 45746 45771 +25
==========================================
+ Hits 43870 43886 +16
- Misses 1876 1885 +9 ☔ View full report in Codecov by Sentry. |
Update self hosted dependency Co-authored-by: GitHub Action <[email protected]>
* fix cls by adding the videoEl without a source * adapt the video hover and in view port play * Remove no-lazy and return earlier * Fix linting issue * Consolidate duplicated logic * Move functions into the init function * Move root margin and use optional chaining * Only query for the videoEl once
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We're seeing the video controls display and autoplay not working on the marquee in this latest PR on bacom. Reproducible on all browsers.
Producible on: https://business.stage.adobe.com/products/real-time-customer-data-platform/rtcdp.html?milolibs=stage
Not reproducible on main: https://business.adobe.com/products/real-time-customer-data-platform/rtcdp.html
cc @mokimo @JasonHowellSlavin @Brandon32
Reverted
common base root URLs
Homepage : https://www.stage.adobe.com/
BACOM: https://business.stage.adobe.com/fr/
CC: https://www.stage.adobe.com/creativecloud.html
Blog: https://blog.stage.adobe.com/
Acrobat: https://www.stage.adobe.com/acrobat/online/sign-pdf.html
Milo: