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

Home page - Page locations #4059

Closed
ahukkanen opened this issue Jul 28, 2016 · 1 comment
Closed

Home page - Page locations #4059

ahukkanen opened this issue Jul 28, 2016 · 1 comment
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Enhancement A need for something new.
Milestone

Comments

@ahukkanen
Copy link
Contributor

Currently it is not possible (as far as I know) to set alternative page locations for the home page. This can be quite confusing in some occasions and definitely a feature that is often needed when updating old websites. After the launch, we might have multiple pages that no longer exist but we want to 301 redirect those to the home page. Currently the fastest way to do this is through manually modifying the .htaccess file.

So, it would be highly convenient if we could do this directly from the UI simply by opening the "page locations" settings for the home page. Is there some reason why this has been disabled for the home page?

Obviously, we cannot change the primary canonical URL for the home page since that would possibly make the site inaccessible. But alternative URLs for the home page should not have any side effects, right?

Some other thoughts on this matter:
https://www.concrete5.org/community/forums/5-7-discussion/301-redirect-to-homepage-in-5.7/
#350 (comment)

I'm sure some other people have also bumped into the situation where they end up searching for the missing "Page Location" section from the settings menu. I also once went deeper into the code for like 1 hour just to figure out it was disabled for the home page.

@aembler aembler added Type:Enhancement A need for something new. Status:Available Reviewed issue, it’s real, we’d review a pull request. priority:like to have labels Jul 28, 2016
@aembler aembler added this to the 8.0 milestone Jul 28, 2016
@aembler
Copy link
Member

aembler commented Jul 28, 2016

Yeah, I agree – I imagine the reason for the limitation is simply because you can't give the home page a different canonical url and we just wrapped the entire thing in a greedy if statement.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Status:Available Reviewed issue, it’s real, we’d review a pull request. Type:Enhancement A need for something new.
Projects
None yet
Development

No branches or pull requests

2 participants