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.

Sign up for GitHub

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

Jump to bottom

news.adobe.com to hlx-5 #40

Closed
wants to merge 17 commits into from
Closed

news.adobe.com to hlx-5 #40

wants to merge 17 commits into from

Conversation

mokimo
Copy link

@mokimo mokimo commented Dec 10, 2024 •
edited
Loading

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

Test URLs:

Ruchika4 and others added 17 commits May 28, 2024 18:00
@Ruchika4
Fix lint issues in fallback and test runner config files (#23)
82309ca 
* Update run-lint.yaml

* Update fallback.js

* Update web-test-runner.config.js

* update package.json
@Ruchika4
Update config.json (#25)
9dc6dbd
@Ruchika4
add sitemap url (#27)
e010efe
@Ruchika4
Update helix-sitemap.yaml (#28)
afe8ab1
@Ruchika4
Update helix-query.yaml
56ab72c
@Ruchika4
Merge pull request #32 from adobecom/sitemap-stage
1d1529d 
Update helix-query.yaml
@Ruchika4
Update scripts.js
d475ed9
@Ruchika4
Merge pull request #33 from adobecom/lana
e3cfcf7 
MWPW-159080-Add lana logging
@Ruchika4
Update .hlxignore
ff6f051
@Ruchika4
Merge pull request #35 from adobecom/hlxignore
8879530 
Update .hlxignore
@Ruchika4
Update .hlxignore
341551a
@Ruchika4
Merge pull request #36 from adobecom/hlxignore
88f37e1 
Update .hlxignore
@Ruchika4
update favicon
3fb740f
@Ruchika4
incorporate review comments
bba6ad4
@Ruchika4
Update favicon.webmanifest
5b071d2
@Ruchika4
Merge pull request #38 from adobecom/favicon
9e69b6e 
MWPW-162067- Update favicon for project red
@mokimo
news.adobe.com to hlx-5
a8faf8c
@aem-code-sync AEM Code Sync
Copy link

aem-code-sync bot commented Dec 10, 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

@aem-code-sync AEM Code Sync
Copy link

aem-code-sync bot commented Dec 10, 2024 •
edited
Loading

Page Scores Audits Google
📱 /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
📱 /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI
🖥️ /?martech=off PERFORMANCE A11Y SEO BEST PRACTICES SI FCP LCP TBT CLS PSI

@Ruchika4
Copy link
Collaborator

There were some sync issue between stage and main. And in order to fix it this PR got some unwanted commits. Creating a copy of this PR here #41 and closing this one cc @mokimo

@Ruchika4 Ruchika4 closed this Dec 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mokimo @Ruchika4