8000 [Release] Stage to Main by milo-pr-merge[bot] · Pull Request #4345 · 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 #4345

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 17 commits into from
Jun 11, 2025
Merged

[Release] Stage to Main #4345

merged 17 commits into from
Jun 11, 2025

Conversation

milo-pr-merge[bot]
Copy link
Contributor
@milo-pr-merge milo-pr-merge bot commented Jun 9, 2025

common base root URLs

Homepage : https://www.stage.adobe.com/
BACOM: https://business.stage.adobe.com/fr/
CC: https://www.stage.adobe.com/creativecloud.html
Blog: https://blog.stage.adobe.com/
Acrobat: https://www.stage.adobe.com/acrobat/online/sign-pdf.html

Milo:

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

GNav Test URLs

Gnav + Footer + Region Picker modal:

Thin Gnav + ThinFooter + Region Picker dropup:

Localnav + Promo:

Sticky Branch Banner:

Inline Branch Banner:

Blog

RTL Locale

hit11757 and others added 12 commits June 9, 2025 18:13
Fixing replace check for test url
* MWPW-170229: Show action scroller nav on resize

* MWPW-170229: Debounce scroll and resize events
* MWPW-170805 initial refactoring of seo checks.

* MWPW-170805 update the links method to cache its result

* MWPW-170805 allow for document to be passed

* update preflight

* update performance checks

* update assets checks

* don't pass url if not necessary to assets

* cache image result

* import mini-preflight in utils

* hotfix - seo title typos

* hotfixes

* hotfix

* hotfix

* Removed unnecessary comments

* fixed linter errors

* removed old performance panel unit tests

* Mwpw 170378 preflight assets unit test  (#4131)

* added unit tests for assets tab in preflight

* fixed linting

* centralized checks imports

* added unit tests

---------

Co-authored-by: Victor Hargrave <vhargrave@adobe.com>
Co-authored-by: Sino Kholkhojaev <132879006+skholkhojaev@users.noreply.github.com>
* 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>

* [Release] Stage to Main (#3817)

* ENB-7820: Add values for event95 (#3782)

* Add values for event95

* Fix failing test case

* Remove spaces

* Minimal Impact: Add extra key for  other features

* Minimal Impact: change function name

* [MWPW-168691] - Aside cta shrink JP (#3785)

* [MWPW-168691] cta shrink fix

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] update

* [MWPW-168691] updated solution

* Revert "[MWPW-168691]"

This reverts commit a70f9ca.

Revert "[MWPW-168691] update"

This reverts commit 6aac2bc.

Revert "[MWPW-168691] update"

This reverts commit c0c6a34.

Revert "[MWPW-168691] update"

This reverts commit 017edf0.

Revert "[MWPW-168691] update"

This reverts commit 219e5fd.

Revert "[MWPW-168691] update"

This reverts commit a7df2b5.

Revert "[MWPW-168691] update"

This reverts commit 299a7d7.

Revert "[MWPW-168691] update"

This reverts commit 6709515.

Revert "[MWPW-168691] cta shrink fix"

This reverts commit c90eea6.

* MWPW-168691: Prevent cta jp word wrap on large screens

* MWPW-168691: Fix notification jp cta

* MWPW-168691: Hide wbr cta for tablet and desktop

---------

Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>

* MWPW-169371 : Adding countryCode config to UNav (#3793)

* Adding countryCode config to UNav

* Updating countryCode for uk

* Updating for upperCase

* Passing countryCode according to MAS mapping for UCv3 cart

---------

Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>

* MWPW-167716 Remove await from both gnav and georouting (#3680)

Made georouting and globalnavigation non blocking

* Lnav title and headline text overflow support (#3781)

* Lnav title text overflow support

* Fix for chevron icon top position

* Adding white space break for localnav menus

* Adding white space break for localnav menus

* Adding white space break for localnav list menus

* Inclufing white spaces to non menu items

* [MWPW-169500] SEO- Links check failing breaks Preflight (#3803)

fixed preflight seo links check

* MWPW-169732 Fetch resources from adobe.com/federal instead of https://main--federal--adobecom.aem.live/federal (#3813)

* on prod federal content for the standalone gnav and footer needs to come from

* handled situations where we're stage and when we're neither stage nor prod

* MWPW-169416 [MEP] stop treating remove action differently in preview (#3789)

* truly remove elements, even in preview

* unit test update

* fix for MWPW-169416, new remove action in MEP

* lint clean-up

---------

Co-authored-by: John Pratt <jpratt@adobe.com>

---------

Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Raghav Sharma <118168183+sharmrj@users.noreply.github.com>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>

* MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3824)

MWPW-167191 Removed an unnecessary css url() that was causing a console error (#3630)

* updated an svg inside a url() in the globalnav css to have data:image/svg+xml;utf8 inorder to prevent a console error

* changed a ; to a ,

* Removed the svg as it was unnecessary

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

* MWPW-162398 draft

* MWPW-162398 enable mas preview

- getSettings will consider preview attribute of mas-commerce-service presence as the fact it should be using preview,
- if preview is on, then using mas' previewFragment client library to act like adobeio, but on browser,
- surface used (for stuff like placeholders) is either computed depending on wcs api key, or coming from preview parameter value,
- is url is .page, or www.stage, then preview is set by default,
- preview configuration can be overriden with masPreview parameter (url or local storage) set to off for disabling, or a value to use another surface (if api key is not corresponding to a registered surface)

* some fixes

* some other fixes

* some doc and changes

* small fixes

* Reduce gnav lana sampleRate to 0.01% (#4150)

* review comment

* Revert "Improve carousel accessibility (#4103)" from prod (#4195)

Revert "Improve carousel accessibility (#4103)"

This reverts commit 34d2016.

* some fixes

* review comments

---------

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>
Co-authored-by: Swati Mukherjee <swati020494@gmail.com>
Co-authored-by: Dušan Kosanović <dusan.kosanovic@hitthecode.com>
Co-authored-by: Ratko Zagorac <ratkozagorac@gmail.com>
Co-authored-by: sonawanesnehal3 <152426902+sonawanesnehal3@users.noreply.github.com>
Co-authored-by: Snehal Sonawane <sonawane@snehals-mbp.corp.adobe.com>
Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
Co-authored-by: Bandana Laishram <bandanalaishram@gmail.com>
Co-authored-by: Robert Bogos <146744221+robert-bogos@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: John Pratt <jpratt@adobe.com>
Co-authored-by: Chris Peyer <chrischrischris@users.noreply.github.com>
* added copying to clipboard

* lint fix

* moved to a separate module

* used getMetadata

* added a tooltip

* fixed brackets

* added ClipboardItem

* updated tests

* replaced the icon

* updated unit test
#4273)

* first commit

* Enhance path normalization to include localhost and simplify previous page match logic

* Refactor checkForPreviousPageMatch to accept referrer as a parameter and add comprehensive tests

* Remove commented suggestion for error handl
8000
ing

* Refactor checkForPreviousPageMatch to use lastVisitedPage parameter for improved clarity
* Exposing close gnav function for clients

* Lint fix

* Add a check if the gnav is in open state before calling toggle
* MWPW-173759: Unordered list styling for merch cards

* style specificity fix
* Initial checkin. Working state.

* Move replacecell handling to utils from fragments

* Fix: Replace hardcoded string with constant for replacecell handling

* Refactor: Replace constant OVERRIDE_AUTO_INLINE with inline string in modal handling

* Unit testing.

* Unit testing.

* Removed unnecessary test statement.

---------

Co-authored-by: Vivian A Goodrich <vgoodric@adobe.com>
Co-authored-by: Bozo Jovicic <bozo@hitthecode.com>
@milo-pr-merge
Copy link
Contributor Author
milo-pr-merge bot commented Jun 9, 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 Jun 9, 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
@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/ 9E88 ec096708b2710942ce7e20cef2d905218b3ae22f/libs/features/mas/src/global.css.js


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

https://github.com/adobecom/milo/blob/ec096708b2710942ce7e20cef2d905218b3ae22f/libs/features/mas/src/mas-commerce-service.js


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

https://github.com/adobecom/milo/blob/ec096708b2710942ce7e20cef2d905218b3ae22f/libs/features/mas/src/settings.js


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

https://github.com/adobecom/milo/blob/ec096708b2710942ce7e20cef2d905218b3ae22f/libs/features/mas/test/settings.test.js

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 June 11, 2025 01:21 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge June 11, 2025 04:25 — with GitHub Actions Inactive
@aem-code-sync aem-code-sync bot temporarily deployed to adobeprivacyfederal June 11, 2025 07:04 Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge June 11, 2025 08:22 — with GitHub Actions Inactive
@SilviuLCF SilviuLCF self-requested a review June 11, 2025 08:27
Copy link
@SilviuLCF SilviuLCF left a comment

Choose a reason for hiding this comment

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

BLOG regression - PASS

@mokimo mokimo temporarily deployed to milo_pr_merge June 11, 2025 09:18 — with GitHub Actions Inactive
@narcis-radu narcis-radu requested a review from a team June 11, 2025 10:11
@aem-code-sync aem-code-sync bot temporarily deployed to restrict-branch June 11, 2025 10:30 Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge June 11, 2025 12:35 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge June 11, 2025 16:21 — with GitHub Actions Inactive
@Dli3 Dli3 temporarily deployed to milo_pr_merge June 11, 2025 16:59 — with GitHub Actions Inactive
@milo-pr-merge milo-pr-merge bot merged commit 0a2357a into main Jun 11, 2025
28 of 31 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