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

Support both .aem. and .hlx. - HLX5 Support #621

Merged
merged 2 commits into from
Feb 13, 2025

Conversation

aishwaryamathuria
Copy link
Collaborator

@aishwaryamathuria aishwaryamathuria commented Feb 13, 2025

Description

Upgrading to aem.live from hlx.live as per the product requirements and docs.
This is not meant to be a complete PR but a spike to ensure you have everything you need and should be carefully QA'd

Todo's

  1. This will need proper QA
  2. This will need to get merged
  3. This will need an origin swap on the akamai level to consume aem.page and aem.live
  4. Once we switched the origin, we can delete references to hlx.live or hlx.page

Test URLs

All 4 variations should work: hlx.page aem.page hlx.live aem.live

Before: https://main--cc--adobecom.aem.live/products/photoshop?martech=off&georouting=off
After: https://revert-620-revert-484-hlx-5--cc--adobecom.hlx.live/products/photoshop?martech=off&georouting=off
After-2: https://revert-620-revert-484-hlx-5--cc--adobecom.aem.live/products/photoshop?martech=off&georouting=off

Known issues

@aishwaryamathuria aishwaryamathuria requested a review from a team as a code owner February 13, 2025 13:10
Copy link

aem-code-sync bot commented Feb 13, 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

aem-code-sync bot commented Feb 13, 2025

@spadmasa
Copy link
Collaborator

validation to be done once the pr is in stage

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready for main PR is ready to be merged to main branch ready for stage verified
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants