v3.2.0 contained changes to support support sites which were configured to generated relative links. At the time I implemented these changes we were not making use of the old_paths page redirection functionality. We have since started using this which caused me to realise that the redirect page generation did not work correctly when a site is configured to use relative links.
This manifested in the redirect pages being create in the wrong location in the site hierarchy and their redirect URLs being wrong (pointing to paths which don't exist).
This PR updates the redirect functionality such that it now works correctly when sites are configured to use relative links . I have written some simple redirect unit tests as non existed previously.
Hope this all makes sense. Would be great if someone could give it some 👀 and consider it for inclusion. Very happy to discuss and fettle if required.
Cheers!
Identifying a user need
No UR done. User testing was what actually alerted us to this issue in the first place though, as users were not able to access redirect pages, because they were being generated incorrectly.
User story:
As an author of a tech-docs-gem based site which uses relative links
I want to be able to provide my users with redirect pages
So that I can confidently rename and relocate my pages, without degrading the experience of my users
What’s changed
v3.2.0 contained changes to support support sites which were configured to generated relative links. At the time I implemented these changes we were not making use of the
old_pathspage redirection functionality. We have since started using this which caused me to realise that the redirect page generation did not work correctly when a site is configured to use relative links.This manifested in the redirect pages being create in the wrong location in the site hierarchy and their redirect URLs being wrong (pointing to paths which don't exist).
This PR updates the redirect functionality such that it now works correctly when sites are configured to use relative links . I have written some simple redirect unit tests as non existed previously.
Hope this all makes sense. Would be great if someone could give it some 👀 and consider it for inclusion. Very happy to discuss and fettle if required.
Cheers!
Identifying a user need
No UR done. User testing was what actually alerted us to this issue in the first place though, as users were not able to access redirect pages, because they were being generated incorrectly.
User story: