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 #3187

Merged
merged 5 commits into from
Nov 14, 2024
Merged

[Release] Stage to Main #3187

merged 5 commits into from
Nov 14, 2024

Conversation

bozojovicic and others added 2 commits November 13, 2024 12:06
* MWPW-161289 Catalog CTA IMS country used in commerce calls

* Trigger Build

* MWPW-161289 Catalog CTA IMS country used in commerce calls

* MWPW-161289 Catalog CTA IMS country used in commerce calls - commerce re-generated

* Trigger Build

* Trigger Build

* MWPW-161289 Catalog CTA IMS country used in commerce calls

* MWPW-161289 Catalog CTA IMS country used in commerce calls

* Trigger Build

* Expired test promo fixed

* Expired test promo fixed

* Trigger Build

* Trigger Build

* New commerce build

* Expired test promo rollback

* New commerce build

* Trigger Build

* Trigger Build

* New mas build

* New mas and commerce build

---------

Co-authored-by: Bozo Jovicic <bozo@hitthecode.com>
* Helix 5 support for milo blocks and features

* Fix the failing unit test

* Fix personalization nala tests

* Update libs/blocks/quiz-entry/quiz-entry.css

Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>

* Embed Rares' PR feedback

* Account for authored paths to include either .hlx. or .aem.

* Remove trailing space

* Properly decorate metadata for both hlx4 and hlx5

* Simplify configs

* Add separate privacy ids for hlx and aem

* Adapt codecov patches to hit the threshhold at 90%

---------

Co-authored-by: Brad Johnson <fullcolorcoder@gmail.com>
@milo-pr-merge
Copy link
Contributor Author

milo-pr-merge bot commented Nov 13, 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 Nov 13, 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 Nov 13, 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

@aem-code-sync aem-code-sync bot temporarily deployed to resolve-stage-conflicts November 13, 2024 12:25 Inactive
@mokimo mokimo changed the base branch from main to newsletter-modal November 13, 2024 14:06
@mokimo mokimo changed the base branch from newsletter-modal to main November 13, 2024 14:06
@JackySun9
Copy link
Contributor

JackySun9 commented Nov 13, 2024

@mokimo could you help check this for CAAS: https://stage--milo--adobecom.hlx.live/fragments/caas/test-pages/textcardcollection ,images are missing

@tedAir tedAir requested review from tedAir and removed request for a team, nishantka and sonawanesnehal3 November 13, 2024 21:36
Copy link
Contributor

@JackySun9 JackySun9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

hold this for CAAS with helix 5, tracking at https://jira.corp.adobe.com/browse/MWPW-162425

@mokimo
Copy link
Contributor

mokimo commented Nov 14, 2024

Detected the issue, this would just be happening on milo .page and .live pages, so ideally it wouldn't have affected production #3195

* add nala folder to mep folder

* add observer for the body element

* rename once

* add css

* new folder

* same name, same file, different folder

* cleanup

---------

Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: vgoodric <vgoodric@adobe.com>
@milo-pr-merge milo-pr-merge bot merged commit 6fe3d53 into main Nov 14, 2024
18 of 19 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