-
Bug
-
Resolution: Fixed
-
Major
-
None
-
None
-
None
-
Empty show more show less
-
Yes
-
DevX 30
-
3
Step to reproduce:
- Using hello-saas repo here
- Add new page travel with template basic-with-site to use site aware render type
- Open travel page detail and see the iframe load correct path:Â Â
http://cooltravel.com/travel?mgnlPreview=false&mgnlChannel=desktop
- Now change of travel site routeTemplate here  from @path to @name as below
routeTemplate: '/{language}{{@name}}'
- See iframe path has website workspace as below
http://cooltravel.com/website/travel?mgnlPreview=false&mgnlChannel=desktop
Expectation: Do not include website in url. Should be like this: http://cooltravel.com/travel?mgnlPreview=false&mgnlChannel=desktop
Â
Hint: May be missing switch case for NAME in DefaultSpaRouter. This is example of Path case in this PR: https://git.magnolia-cms.com/projects/MODULES/repos/pages/pull-requests/486/diff#magnolia-spa-norsu-rendering/src/main/java/info/magnolia/pages/norsu/routing/DefaultSpaRouter.java
Acceptance criteria
- is related to
-
PAGES-948 [Norsu] @path value in routeTemplate should not include workspace
- Closed
- to be documented by
-
PAGES-1094 DOC: Remove prefix website when using @name in routeTemplate of site norsu definition
- Closed
1.
|
Implement | Completed | Canh Nguyen | |||||||||
2.
|
Review | Closed | Lam Nguyen Bao | |||||||||
3.
|
piQA | Closed | Oanh Thai Hoang |
|
||||||||
4.
|
QA | Completed | Oanh Thai Hoang |
|