-
Notifications
You must be signed in to change notification settings - Fork 190
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
Sync with Stage #4266
Conversation
* 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)
* [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
* 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
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
Commits
|
There was a problem hiding this 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
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
Unexpected console statement.
Line 738 in 8e28f72
console.log(err); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
File ignored because of a matching ignore pattern. Use "--no-ignore" to override.
475e999
into
korea-free-trial-restrict
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