8000 Sync with Stage by narcis-radu · Pull Request #4266 · adobecom/milo · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content

Sync with Stage #4266

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 31 commits into from
May 29, 2025
Merged

Sync with Stage #4266

merged 31 commits into from
May 29, 2025

Conversation

narcis-radu
Copy link
Contributor
@narcis-radu narcis-radu commented May 29, 2025
  • Add your
  • Specific
  • Features or fixes

Resolves: MWPW-NUMBER

Test URLs:

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

akanshaa-18 and others added 30 commits May 21, 2025 09:32
* Missing/Mismatched Attributes

* Missing/Mismatched Attributes

* Adding requested changes

* Fixing Locale Issue

* Fixing Locale Issue

* Updating helpers.md

* Fixing Errors

* Fixing Errors

* Fixing Errors

* Fixing Errors

* Fixing Errors

* Updated Attributes + Deletions

* Updated Attributes + Deletions

* Updated Attributes + Deletions

* Fixing Errors

* Fixing Errors

* Fixing Errors

* Fixing setEvent - secondVisits

* Fixing setEvent - secondVisits

* Fixing consent issue for secondHit

* Fixing acrobatSecondVisit

* Fixing locale Bug

* Fixing errors

* Fixing errors

* Fixing errors

---------

Co-authored-by: Akansha Arora <>
…tage (#4124)

* added filtering of page manifests by filter params

* added filtering of page manifests by filter params

* conf

* first commit

* temporarily disabled ajo and placeholder manifestsrc options

* MWPW-173501 [MMM][MEP] Add geo filter to MMM Target cleanup report (#4172)

* enabled geo filter on mmm-target-cleanup

* change naming for base vs targetCleanUp vars

* MWPW-173732 [MMM][MEP] Add contextually relevant button in Target Metadata Lookup Report (#4192)

* first commit

* add button selector for .instructions text block

* Refactor metadata URLs in createMetadataLookup function for improved structure and clarity

* Add TARGET_METADATA_OPTIONS for improved metadata handling in createMetadataLookup

* Update label text for last seen manifests dropdown for mmm and mep-target-inactivity

* Refactor metadata lookup to use updated search function for selectedRepo

---------

Co-authored-by: Vivian A Goodrich <vgoodric@adobe.com>

* [MWPW-173083] [MEP][MMM] Add history from MMM to MEP button in page (#4162)

* Initial checkin. Good state.

* Naming conventions. Fix sticky marquee logic.

* Variant name update.

* Alignment fix.

* Update with hide at action area logic.

* Updated as per PR comment.

* Marquee updates, first pass.

* Var1 marquee button rework.

* Possible custom-hide fix.

* Initial checkin. Fetch manifests API. Generate HTML.

* Headers, divider logic, change events, hide selected, 'non' select option logic.

* Update to duplicate manifest logic.

* Remove console log.

* Linting.

* Unit test update.

* Label/select rename. Don't update button for hidden experiences. Default MMM to none.

* Hidden option button update fix.

* Unit Testing.

* Update libs/features/personalization/preview.js

* Remove placeholder domain.

* Update manifest option titles for clarity in preview.js

* Update option title for clarity in getManifestListDomAndParameter function

---------

Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <vgoodric@adobe.com>

---------

Co-authored-by: Denys Fedotov <dfedotov@Denyss-MacBook-Pro.local>
Co-authored-by: viloria <viloria@adobe.com>
Co-authored-by: Jan Ivan Viloria <158211769+ivanvatadobe@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <vgoodric@adobe.com>
Co-authored-by: Dave Linhart <132396886+AdobeLinhart@users.noreply.github.com>
Co-authored-by: Vivian A Goodrich <101133187+vgoodric@users.noreply.github.com>
… has not loaded (Desktop & Mobile) (#4137)

* Parallelize the network calls required before the gnav becomes visible

* corrected the logic for loading base styles for non-milo contexts

* load aside.js in parallel if necessary

* made the logic inside getGnavSource a little more fault tolerant

* Fixed an issue with loading the dark mode gnav (standalone)

* changed the gnav tests to reload the global-navigation.js module anew on each test to work correctly with the performance optimizations

* Fixed a test for localnav inside bootstrapper.test.js

* ignore eslint error on a test

* added error handling to the style loading logic in gnav

* changed error handling for loading gnav styles from throwing an error to rejecting a promise

* removed some redundant code

* changed e.message to e

* implemented the loading state for desktop viewports

* Change animation name as it was the same as one that the unav uses

* implemented the loading state for mobile

* Removed pointer events from the loading state; made the loading state closer to the specs

* Finishing touches

* Don't have a loading state for the legacy mobile gnav

* add the loading state only on async Dropdowns; decorateLocalnav should be inside decorateMenu.then(...)

* put the call to `decorateLocalNavItems` inside `finally` to make sure it runs

* lint fixes

* Sometimes mobileNavCleanup can be null

* Removed an extraneous line

* Cleaning up the merge conflict resolution

* Make sure `transformTemplateToMobile` a
10000
lways returns a function
Added ally fix for GNAV Promo heading text

Co-authored-by: Dev Ashish Sardana <glo77801@adobe.com>
* [MWPW=170714] - section data mobile margin

* [MWPW=170714] - section two up metadata change

* [MWPW-170714] - update mobile gap solution

* [MWPW-170714] - remove media query
* MWPW-170799: Remove external.js and refactor imports.

* fix regressions

* fix regressions

* fix checks

* fix checks

* still regression in variants

* fix regression on ccd ah styles

* disable aggressive PR comments

* remove unused file

* fix regressions

* fix error in doc page

* PR review

* fix regression in variant mappings

* fix plans url

* cleanup constants

* merge stage and fix regressions

* MWPW-164498: Plans Education card

* styles

* plans page update

* heading global styles

* phas

* block

* business plans initial draft

* fix badge

* spacing and growing

* height and styles

* collection styles

* remove whats included from education

* removed whats included in some other way

* Plans students aem mapping

* milo plans page style updates

* removed hardcoded link color

* link fix

* MWPW-173121: ABM label for Plans body prices

* unit test

---------

Co-authored-by: Ilyas Türkben <ilyas@adobe.com>
Co-authored-by: Bozo Jovicic <bozo@hitthecode.com>
* MWPW-170301: Hide cookie banner when modal is open

Revert "MWPW-170301: Keep focus inside the modal"

This reverts commit df02548.

MWPW-170301: Hide cookie banner when modal is open

* MWPW-170301: Move isDeepLink to getModal

* MWPW-170301: Add deepLink property to details
* Breadcrumb last item value as current window url if not a link

* Breadcrumb test case fix

* Breadcrumb Lint fix

* Breadcrumb Lint fix
* show titles on mobile
* add support for rtl for option titles

Resolves: [MWPW-173478](https://jira.corp.adobe.com/browse/MWPW-173478)
…fragment has not loaded (Desktop & Mobile)" (#4210)

Revert "MWPW-167709 [GNAV] [Pre-Load] | (Ghost) State when Mega Menu fragment…"

This reverts commit 02289eb.
* [MWPW-173016] - block korea free trial links/buttons

* [MWPW-173016] - korea text filter added

* [MWPW-173016] - update if

* [MWPW-173016] - code optimization

* [MWPW-173016] - improve import

* [MWPW-173016] - modal check, string check added

* [MWPW-173016] - strings added

* [MWPW-173470] - update logic

* [MWPW-173470] - update logic

* [MWPW-173470] - update logic

* [MWPW-173470] - remove whitespace

* [MWPW-173470] - optimize

* [MWPW-173470] - remove variable

* [MWPW-173016] - optimize code

* [MWPW-173016] - group logic

* [MWPW-173016] - merch solution added

* [MWPW-173016] - merch ref comment added

* [MWPW-173470] - null safe

* [MWPW-173470] - code optimization

* [MWPW-173470] - nav korea restrict coverage

* [MWPW-173470] - fix eslint console error
* MWPW-173239: fallback for addon

* add unit test

* address review comments
* wip

* wip

* wip

* wip

* unit tests + css

* demo page

* updated deps

* tooltip

* tooltip

* Update popover.js

* Update fries.html

* Update fries.html

* cta toggle text

* wip

* updated deps

* wip

* wip

* updated deps

* unit tests

* ut fixes

* updated deps

* ut fix

* Update merch-offer-select.test.js

* NALA fix

* conflicts resolved

* fixes

* text decoration

* yellow color dark theme

* Update fries.html

* remove console log

* font size addon confirmation

* Update merch-card.fries.test.html.js

* dark theme body color

* dark theme

* icon size

* responsiveness

* styles

* body-s color

* Update styles.css

* Update fries.html

* Updated CSS

* color

* updated deps

* resolved conflicts

* css

* increased max count for fries card
* ribbon in-line

* ribbon in-line

* ribbon in-line

* review comment

---------

Co-authored-by: Drashti Modasara <dmodasara@Drashtis-MacBook-Pro.local>
* MWPW-169410: Add download icon to sidekick

* MWPW-169410: Remove title and description to avoid announcment duplication

* MWPW-169410: Change title to Download icon
Updating codeowners for region nav and navigation folder
* Wrapping the test urls in a collapsed section

* Update title
* Update notification z indexes with values

* Update notification.css
)

* Logging only when it is sent as localnav but is not

* Test fix

* Lint fix
* Tab, ShiftTab, Escape fixes for mobile design

* Updated for escape functionality

* accommodating review comments

---------

Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
…with screen reader (#4205)

* Fix for pagecontent reading

* Fix for pagecontent reading

* Fix for pagecontent reading

---------

Co-authored-by: Snehal Sonawane <sonawane@Snehals-MacBook-Pro.local>
* MWPW-171937: fix cta urls for bulk publisher

* MWPW-171937: fix cta urls for bulk publisher

---------

Co-authored-by: Sheridan Sunier <sunier@Sheridans-MacBook-Pro.local>
…& multiple cta reopen fix (#4225)

* MWPW-173888: Appending extra options for relative urls

* keep relative url

* modal reopened logic

* unit tests
Revert "[MWPW-173470] - Korea free trial restrict (#4185)"

This reverts commit 351cfe6.
Copy link
Contributor
aem-code-sync bot commented May 29, 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/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/merch-card.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/merch-icon.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/variants/ccd-slice.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/variants/fries.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/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/variants/fries.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/variants/plans.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/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/src/variants/plans.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/test/hydrate.test.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/test/merch-card.fries.test.html.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/mas/test/merch-offer-select.test.html.js


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

https://github.com/adobecom/milo/blob/8e28f72fd6a99897c2d9186b935edf0a333a4419/libs/features/spectrum-web-components/dist/popover.js


⚠️ [eslint] <no-console> reported by reviewdog 🐶
Unexpected console statement.

console.log(err);

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.

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.

@narcis-radu narcis-radu merged commit 475e999 into korea-free-trial-restrict May 29, 2025
96 of 98 checks passed
@mokimo mokimo temporarily deployed to milo_pr_merge May 29, 2025 16:20 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge May 29, 2025 20:18 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge May 30, 2025 01:17 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge May 30, 2025 04:21 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge May 30, 2025 08:21 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge May 30, 2025 09:17 — with GitHub Actions Inactive
@mokimo mokimo temporarily deployed to milo_pr_merge May 30, 2025 12:33 — with GitHub Actions Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

0