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

[Release] Stage to Main #2893

Merged
merged 5 commits into from
Sep 18, 2024
Merged

[Release] Stage to Main #2893

merged 5 commits into from
Sep 18, 2024

Conversation

mokimo and others added 4 commits September 17, 2024 09:15
* MWPW-155412: Fix video CLS netting +10 lighthouse points (#2724)

* 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

* Add default hash at in the right spot

* Fix the figure/video interaction

* Remove unused export statement

* Keep the previous defaults

* Re-add adding autoplay

* Retain decorateBlockBg logic

* Move autoplay logic to the right location

* Fix linting issues

* Adapt figure tests

* PR Feedback

* Safeguard decorateAnchorVideo

* Remove anchor if neccessary

* Add video test
… using a `full-width` variant (#2861)

* stub no-min-height

* fixed hero has-bg logic
* Upgrade UNav to 1.3

* using await instead of then
* MWPW-158462 [MEP] any-marquee-section simplified selector

* unit test update

* MWPW-158475 [MEP] classes that end in a number are modified when they should not be (#2886)

* unit test update

* MWPW-158475 [MEP] classes that end in a number are modified when they should not be

* unit test repair

* unit test repair

* MWPW-157686 [MEP] Cannot spoof an experience that exists in manifest but not in Target (#2887)

* working so far

* lint update

* update something merging stage did not
@milo-pr-merge
Copy link
Contributor Author

milo-pr-merge bot commented Sep 17, 2024

Testing can start @adobecom/miq-sot @adobecom/bacom-sot @adobecom/homepage-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot

Copy link
Contributor

aem-code-sync bot commented Sep 17, 2024

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

Copy link
Contributor

aem-code-sync bot commented Sep 17, 2024

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

Copy link

codecov bot commented Sep 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.24%. Comparing base (840e037) to head (9323f28).
Report is 54 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #2893      +/-   ##
==========================================
+ Coverage   96.23%   96.24%   +0.01%     
==========================================
  Files         237      237              
  Lines       54242    54219      -23     
==========================================
- Hits        52198    52182      -16     
+ Misses       2044     2037       -7     

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

…2888)

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
@JackySun9
Copy link
Contributor

I saw some error on this page, https://blog.stage.adobe.com/ who can take a look?

@milo-pr-merge milo-pr-merge bot merged commit cc08e6e into main Sep 18, 2024
41 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants