Stage to Main Release

[fullcolorcoder]

Release of the following:

#127
#131
#134
#136
#138
#140
#141
#142
#143

Test URLs:

• Before: https://main--express-milo--adobecom.aem.page/express/?martech=off
• After: https://stage--express-milo--adobecom.aem.page/express/?martech=off

[aem-code-sync]

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

• 005a22c ✅ (latest)
• 73205c2 ✅
• aec1ced ✅
• e5621ea ✅
• fafea77 ✅
• 5e56415 ✅
• 229af1d ✅
• 3366474 ✅
• 407bdfb ✅
• 19ce183 ✅
• 9505ef1 ✅
• 8e2cca3 ✅
• 023892f ✅
• f471c3b ✅
• 7a6fd14 ✅
• 5f1f23b ✅
• b5bf4a1 ✅
• e32d13e ✅
• 781aa72 ✅
• 6813067 ✅
• 940afe9 ✅
• a47d29c ✅
• 68e100a ✅
• 164eab0 ✅
• 36214ca ✅
• 5b6b707 ✅
• 3a283c0 ✅
• 3ea01e7 ✅
• a9d0a4a ✅
• fe21934 ✅

[aem-code-sync]

	Page	Scores	Audits
📱	/express/?martech=off
🖥️	/express/?martech=off

[vhargrave]

For me the faqv2 block looks like it still needs a little work , especially since it's new and should probably be up to snuff when rolling out.

things like setting the innerHTML on line 6 of the JS file to an empty string is bad practice since it's bad for performance and kills any listeners that were previously on links for example, or that whole moving the header out of the block shenanigans on line 14 seems very strange to me. Those are the precise types of antipatterns that I'm hoping Express can now avoid, now that it's on milo.

I'd personally revert that part, and the rest looks pretty good imo.