-
Notifications
You must be signed in to change notification settings - Fork 190
[Release] Stage to Main #4209
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 #4209
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` always 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)
Testing can start @adobecom/bacom-sot @adobecom/creative-cloud-sot @adobecom/document-cloud-sot @adobecom/express-sot @adobecom/homepage-sot @adobecom/miq-sot |
Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch and validate page speed.
|
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.
we do see pricing model issues. they were not loading in cc, product pages. and some flows model was showing incomplete content. more details in PR conversations.
Seems some PR got added late evening which may cause the issues
Observed issue with the pricing model load in CC product pages.
|
@sigadamvenkata @prativas22 |
Hi @3ch023 Yes, we are able to reproduce it in fresh incognito mode for both the above mentioned issues |
Had a call on issues with modals, noting here: @spadmasa @prativas22 I recommend unblocking batch. Another change of behaviour is observed, on Photoshop page, there are multiple modals, one of them uses deeplink to teams. After this PR if you access https://www.adobe.com/products/photoshop.html#mini-plans-web-cta-photoshop-card it will open teams deeplinked modal - I will clarify with PdMs if thats an issue or not, but so far lets proceed with validation. |
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