MWPW-157618: How-to block video support for mp4 and adobetv

[joaquinrivero]

• How-to block video support for mp4 and Adobe TV

Resolves: MWPW-157618

Test URLs:

• https://mwpw-157618--milo--joaquinrivero.hlx.page/

HOW TO

• Before: https://main--dc--adobecom.hlx.page/drafts/rivero/jpg-to-pdf?martech=off
• After: https://main--dc--adobecom.hlx.page/drafts/rivero/jpg-to-pdf?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

Onhover play

• Before: https://main--milo--adobecom.hlx.live/drafts/ruchika/videoplay-onhover?martech=off
• After: https://main--milo--adobecom.hlx.live/drafts/ruchika/videoplay-onhover?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

Onviewport play

• Before: https://main--milo--adobecom.hlx.live/drafts/siva/products/videoplay-onviewport?martech=off
• After: https://main--milo--adobecom.hlx.live/drafts/siva/products/videoplay-onviewport?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

Only desktop (mobile should not load the video for perf reasons)

• Before: https://main--milo--adobecom.hlx.live/drafts/seanchoi/marquee-video?martech=off
• After: https://main--milo--adobecom.hlx.live/drafts/seanchoi/marquee-video?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

Split marquee video tests

• Before: https://main--milo--adobecom.hlx.live/docs/library/kitchen-sink/marquee-split?martech=off
• After: https://main--milo--adobecom.hlx.live/docs/library/kitchen-sink/marquee-split?martech=off&milolibs=mwpw-157618--milo--joaquinrivero
• Before: https://main--cc--adobecom.hlx.page/drafts/00-CC-milo-migration/bugs/document?martech=off
• After: https://main--cc--adobecom.hlx.page/drafts/00-CC-milo-migration/bugs/document?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

CLS

• Before: https://main--milo--adobecom.aem.live/drafts/osahin/magento-commerce-1
• After: https://main--milo--adobecom.aem.live/drafts/osahin/magento-commerce-1?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

BACOM tests

• Before: https://stage--bacom--adobecom.hlx.live/products/real-time-customer-data-platform/rtcdp?milolibs=stage
• After: https://stage--bacom--adobecom.hlx.live/products/real-time-customer-data-platform/rtcdp?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

CC tests

• Before: https://main--cc--adobecom.hlx.live/creativecloud/buy/education
• After: https://main--cc--adobecom.hlx.live/creativecloud/buy/education?martech=off&milolibs=mwpw-157618--milo--joaquinrivero

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.34%. Comparing base (1bdfd4a) to head (6a22349).
Report is 19 commits behind head on stage.

Additional details and impacted files

@@           Coverage Diff           @@
##            stage    #3029   +/-   ##
=======================================
  Coverage   96.34%   96.34%           
=======================================
  Files         243      243           
  Lines       55284    55284           
=======================================
  Hits        53261    53261           
  Misses       2023     2023           

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

[github-actions]

This pull request is not passing all required checks. Please see this discussion for information on how to get all checks passing. Inconsistent checks can be manually retried. If a test absolutely can not pass for a good reason, please add a comment with an explanation to the PR.

[mokimo]

Let's turn this into a real PR and not a draft - we'll definitely need a fix for this.

[ryanmparrish]

I'm seeing some regression issues. Can you take a look and make sure there are no impacting changes to current content.

Before: https://main--milo--adobecom.hlx.page/docs/library/blocks/howto
After: https://mwpw-157618--milo--joaquinrivero.hlx.page/docs/library/blocks/howto

Maybe you could add these to the description and also update the exiting before link to a useful comparison. Thanks

[mokimo]

Lovely, there's some nala tests failing related to videos - however once they are fixed - this looks great! 👍

[aem-code-sync]

	Page	Scores	Audits	Google
📱	/
🖥️	/

[mokimo]

Awesome 👍 Marking this as high priority as this is an existing production issue that we should fix asap after the RCP.
With the label, this will get priority over other PRs and ensure it gets merged with priority by the automation.