Skip to content
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

MWPW-153962: support maslibs approach #28

Merged
merged 7 commits into from
Jul 9, 2024
Merged

Conversation

yesil
Copy link
Collaborator

@yesil yesil commented Jul 8, 2024

This PR replaces the http meta approach with search params in mas.js url for passing the context.

e.g:

https://www.adobe.com/mas/mas.js   // us english no merch card
https://www.adobe.com/mas/mas.js?locale=CH_de&features=merch-card    // swiss german with merch-card

the features=merch-card parameter is to support CCD/CCH use cases, and Milo will keep loading those dependencies separately.

Resolves: MWPW-153962

Milo PR: adobecom/milo#2544

Test URLs:

Copy link

aem-code-sync bot commented Jul 8, 2024

Page Scores Audits Google
/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@yesil yesil changed the title MWPW-153962: support maslibs approacj MWPW-153962: support maslibs approach Jul 8, 2024
@yesil yesil added the do not merge PR should not be merged yet label Jul 8, 2024
@yesil yesil removed the do not merge PR should not be merged yet label Jul 9, 2024
@3ch023 3ch023 merged commit 0f2c17a into adobecom:main Jul 9, 2024
5 checks passed
rohitsahu pushed a commit that referenced this pull request Jul 10, 2024
…into rosahu/MWPW-147172

* 'rosahu/MWPW-147172' of https://github.com/adobecom/mas:
  feat(MWPW-142267): Merch What's Included and Merch Mnemonic List (TwP) (#4)
  MWPW-153962: update /lib to /libs (#30)
  MWPW-153658 audit script (#25)
  MWPW-153962: support maslibs approach (#28)
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.

3 participants