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

Merged
merged 16 commits into from
May 22, 2025
Merged

[Release] Stage to Main #4209

merged 16 commits into from
May 22, 2025

Conversation

milo-pr-merge[bot]
Copy link
Contributor
@milo-pr-merge milo-pr-merge bot commented May 21, 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

akanshaa-18 and others added 11 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` 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)
@milo-pr-merge
Copy link
Contributor Author
milo-pr-merge bot commented May 21, 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 May 21, 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

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.

9E7A
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.

F438

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

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.

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

@3ch023 @Roycethan @aishwaryamathuria

@prativas22
Copy link
prativas22 commented May 22, 2025

Observed issue with the pricing model load in CC product pages.

  1. on direct access to the URL , it doesn't load the pricing modal
    https://www.stage.adobe.com/products/photoshop.html#mini-plans-web-cta-photoshop-card
    https://www.stage.adobe.com/creativecloud.html#mini-plans-web-cta-acrobat-pro-card
    image

  2. Intermittently observed that the pricing modal loads with broken ,on click Free trail CTA
    https://www.stage.adobe.com/creativecloud.html
    image
    image

@3ch023
Copy link
Contributor
3ch023 commented May 22, 2025

@sigadamvenkata @prativas22
are you able to reproduce in incognito?
it seems like the issue is caused by browser cookies reaching the limit (happens for adobe employees since we are using multiple accounts)
image
https://commerce.adobe.com/store/segmentation?cli=mini_plans&ctx=if&co=US&lang=en&ms=COM&ot=TRIAL&pa=phsp_direct_individual&rtc=t&lo=sl&af=uc_new_user_iframe%2Cuc_new_system_close

If I try in incognito, modals load fine for me:
image

@prativas22
Copy link
prativas22 commented May 22, 2025

Hi @3ch023

Yes, we are able to reproduce it in fresh incognito mode for both the above mentioned issues
image
image

on direct URL access with parameter , it doesn't load
image

CC: @spadmasa @sigadamvenkata

@3ch023
Copy link
Contributor
3ch023 commented May 22, 2025

Had a call on issues with modals, noting here:
root cause: when there are multiple cta with same modal hash on a page, code opens all these modals - this is an issue that needs to be fixed, however it is not caused by the new code.
the reason that issue is surfaced is because Stage content differs from Prod content.
new 3in1 feature is enabled on stage, and issue is only reproducible on 3in1 modals.
Legacy modals on prod will keep opening as expected even with merge of this PR - tested that on a separate draft page.

@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.

@milo-pr-merge milo-pr-merge bot merged commit 157b88b into main May 22, 2025
42 of 43 checks passed
@milo-pr-merge milo-pr-merge bot temporarily deployed to milo_pr_merge May 22, 2025 12:41 Inactive
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