Details
-
Bug
-
Resolution: Fixed
-
Major
-
None
-
None
-
None
Description
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
Checklists
Acceptance criteria
Attachments
Issue Links
- 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 |
|