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

Adding "Simple Address" to page/experience seems to break routing #49

Open
daniel-isaacs opened this issue Oct 22, 2024 · 2 comments
Open
Assignees
Labels
bug Something isn't working dxp-clients Needs to be addressed by a new release of the NPM packages for Optimizely SaaS CMS

Comments

@daniel-isaacs
Copy link

Defining a "Simple Address" value on a Page/Experience breaks routing, causes 404 on both the Simple Address URL and the original URL:

image

To reproduce on existing site:

  • Create page or experience
  • Browse to page on front-end site to confirm it loads correctly
  • Add "Simple Address" to the content in the CMS
  • Browse to original page URL on front-end site and browse to simple address URL on front-end site

Result:

  • 404 on both URLs

Expected Result:

  • Both URLs successfully load (with the same page content)
  • Page loaded via Simple Address URL has canonical URL pointing at full URL
@remkoj remkoj added the bug Something isn't working label Nov 1, 2024
@MineshS
Copy link

MineshS commented Nov 29, 2024

Following Issue

@remkoj remkoj self-assigned this Dec 3, 2024
@remkoj
Copy link
Collaborator

remkoj commented Dec 3, 2024

Fixed in "Style Enhancements" branch, by upgrading packages to ^3.2

@remkoj remkoj added the dxp-clients Needs to be addressed by a new release of the NPM packages for Optimizely SaaS CMS label Dec 3, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working dxp-clients Needs to be addressed by a new release of the NPM packages for Optimizely SaaS CMS
Projects
None yet
Development

When branches are created from issues, their pull requests are automatically linked.

3 participants