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-161280: Update setLibs so it points to .adobe.com/libs #115

Merged
merged 1 commit into from
Nov 5, 2024

Conversation

zagi25
Copy link
Collaborator

@zagi25 zagi25 commented Nov 1, 2024

Milo libs:

  • partners.adobe.com -> milo.adobe.com
  • partners.stage.adobe.com -> milo.stage.adobe.com
  • main--dme-partners--adobecom.hlx.live/page -> main--milo--adobecom.hlx.live/libs
  • stage and featured branches -> stage--milo--adobecom.hlx.live/libs
  • milolibs query param works only on .hlx.{page/live} links

Resolves: MWPW-161280

Before: https://stage--dme-partners--adobecom.hlx.live/channelpartners/
After: https://mwpw-161280-milolibs--dme-partners--adobecom.hlx.live/channelpartners/

Copy link

aem-code-sync bot commented Nov 1, 2024

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

aem-code-sync bot commented Nov 1, 2024

Page Scores Audits
📱 /channelpartners/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS
🖥️ /channelpartners/ PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS

@codecov-commenter
Copy link

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.50%. Comparing base (05344b6) to head (9c1438f).

Additional details and impacted files
@@            Coverage Diff             @@
##            stage     #115      +/-   ##
==========================================
- Coverage   82.64%   82.50%   -0.14%     
==========================================
  Files          10       10              
  Lines        3249     3253       +4     
==========================================
- Hits         2685     2684       -1     
- Misses        564      569       +5     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

const partnerBranch = hostname.startsWith('main') ? 'main' : 'stage';
const branch = new URLSearchParams(search).get('milolibs') || partnerBranch;
if (branch === 'local') {
return 'http://localhost:6456/libs';
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

why port 6456? never seen this port before

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Ben-Zahler this is the default milo port for when you want to get milo code from your machine so you can debug it. Milo repo has command npm run libs, this will spin up server on port 6456 and when you set query param milolibs=local in your url, you fetch milo code from that server and any local changes that you make to milo repo will be reflected on your page.

@zagi25 zagi25 merged commit 641146a into stage Nov 5, 2024
8 checks passed
@zagi25 zagi25 deleted the MWPW-161280-milolibs branch November 5, 2024 08:40
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.

4 participants