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

[LOC] MWPW-166758: Support for hlx and aem urls for fragments and url validations #3578

Merged
merged 7 commits into from
Jan 29, 2025

Conversation

Copy link
Contributor

aem-code-sync bot commented Jan 29, 2025

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

@saurabhsircar11 saurabhsircar11 added trivial PR doesn't require E2E testing by a reviewer zero-impact labels Jan 29, 2025
@nkthakur48
Copy link
Contributor

@saurabhsircar11 As discussed, please ensure that tenants which are not on hlx5 should have URL pattern matching .hlx.page whereas tenants which are on hlx5 could have URL patten matching .hlx.page and .aem.page for preview URLs

@saurabhsircar11
Copy link
Contributor Author

@saurabhsircar11 As discussed, please ensure that tenants which are not on hlx5 should have URL pattern matching .hlx.page whereas tenants which are on hlx5 could have URL patten matching .hlx.page and .aem.page for preview URLs

Added the old logic for .hlx pages.

@nkthakur48 nkthakur48 merged commit 0c52a04 into adobecom:milostudio-stage Jan 29, 2025
9 of 10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
trivial PR doesn't require E2E testing by a reviewer zero-impact
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants