8000 [Release] Stage to Main by milo-pr-merge[bot] · Pull Request #3772 · adobecom/milo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

[Release] Stage to Main #3772

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

Merged
merged 11 commits into from
Mar 6, 2025
Merged

[Release] Stage to Main #3772

merged 11 commits into from
Mar 6, 2025

Conversation

Deva309 and others added 9 commits March 4, 2025 08:45
…e right side of GNAV. (#3728)

* Added default case adobe 'A' logo on right side for GNAV(Adobe logo block)

* Added default case adobe 'A' logo on right side for GNAV(Adobe logo block)

---------

Co-authored-by: Dev Ashish Sardana <glo77801@adobe.com>
… an iFrame (#3564)

* added 3-in-1 modal

* moved event listener

* updated tests

* addressed comments

* fixed unit test

* moved constants.js into mas.js

* added cli param
* replace overflow x with contain layout

* add a fix for breaking mobile gnav

* move fix to existing media block
* Revert "MWPW-140452 - Icon authoring in milo using the federal repo a… (#3357)

Revert "MWPW-140452 - Icon authoring in milo using the federal repo and individual SVG assets (#3259)"

This reverts commit 81a5770.

* [Release] Stage to Main (#3497)

MWPW-165774 [Mobile-GNAV] page is not scrollable in live page (#3495)

* check for new nav when disabling ios scroll

* shortened the check from the previous commit

Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>

* Initial checkin.

* Remove white space.

---------

Co-authored-by: milo-pr-merge[bot] <169241390+milo-pr-merge[bot]@users.noreply.github.com>
Co-authored-by: Okan Sahin <39759830+mokimo@users.noreply.github.com>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
…3720)

* load georoutingv2.json in parallel with georoutingv2.js

* remove performance marks

* fixed unit tests

* Small refactoring

* fallback for the jsonPromise

* fix unit tests
* [MWPW-167749] table h header replaced

* [MWPW-167749] optimization

* [MWPW-167749] naming changes

* [MWPW-167749] optimization

* [MWPW-167749] code optimization

* [MWPW-167749] col role removal

* [MWPW-167749] code style update

* [MWPW-167309] code improvement

* [MWPW-167309] - header role paragraph
* MWPW-166176 - Stop initial tabs click propagation

* PR feedback

---------

Co-authored-by: Ryan Clayton <rclayton@adobe.com>
@milo-pr-merge
Copy link
Contributor Author
milo-pr-merge bot commented Mar 4, 2025

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

Copy link
Contributor
aem-code-sync bot commented Mar 4, 2025

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 Mar 4, 2025
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
Contributor
@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Remaining comments which cannot be posted as a review comment to avoid GitHub Rate Limit

eslint

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-quantity-select.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-secure-transaction.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-sidenav.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-stock.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-subscription-panel.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-twp-d2p.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/deps/mas/merch-whats-included.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/dist/mas.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/src/buildCheckoutUrl.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/src/checkout-mixin.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/src/checkout.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/src/constants.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/src/external.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/src/mas.js


⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

https://github.com/adobecom/milo/blob/005379eabb39386f5335c7f25fabc859d736d726/libs/features/mas/test/build-checkout-url.test.js

github-actions[bot]
< 10000 div data-view-component="true" class="TimelineItem-body d-flex flex-column flex-md-row flex-justify-start">
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

@mokimo mokimo temporarily deployed to milo_pr_merge March 5, 2025 04:18 — with GitHub Actions Inactive
Copy link
@sigadamvenkata sigadamvenkata left a comment

Choose a reason for hiding this comment

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

There are many areas broken in android mobile. there is huge white space introduced. PFA
Also Gnav has broken. PFA
reason PR: #3706

@sigadamvenkata
Copy link

image
image

@mokimo mokimo temporarily deployed to milo_pr_merge March 5, 2025 08:20 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge March 5, 2025 09:16 — with GitHub Actions Inactive
Revert "replace overflow x with contain layout (#3706)"

This reverts commit 0f54535.
Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

Copy link
Contributor

Choose a reason for hiding this comment

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

⚠️ [eslint] reported by reviewdog 🐶
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.

@mokimo mokimo temporarily deployed to milo_pr_merge March 5, 2025 12:30 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge March 5, 2025 16:24 — with GitHub Actions Inactive
@Roycethan
Copy link

Fixed post revert PR: #3776

@skumar09 skumar09 self-requested a review March 6, 2025 00:30
@mokimo mokimo temporarily deployed to milo_pr_merge March 6, 2025 01:11 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge March 6, 2025 04:17 — with GitHub Actions Inactive
@milo-pr-merge milo-pr-merge bot merged commit 8dbad8c into main Mar 6, 2025
35 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.

0