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

Prevent duplicate content for alternative home page URL #1134

Closed
dav-is opened this issue Feb 25, 2017 · 5 comments · Fixed by flarum/tags#64
Closed

Prevent duplicate content for alternative home page URL #1134

dav-is opened this issue Feb 25, 2017 · 5 comments · Fixed by flarum/tags#64
Assignees
Milestone

Comments

@dav-is
Copy link

dav-is commented Feb 25, 2017

www.google.com/search?q=site%3Adiscuss.flarum.org/all

This is an issue on a few levels. Why can users access https://discuss.flarum.org/all ? I'm told that there's an infinite number of pages too. Like https://discuss.flarum.org/all?page={x}.

@dav-is dav-is changed the title NoJs Page CEO issues NoJs Page SEO issues Feb 25, 2017
@franzliedke
Copy link
Contributor

There is no NoJS content in these search results (anymore?), as far as I can tell.

As for the /all page - that should only be indexed when Flarum's "Home page" setting is set to "Tags", right? When not, it should have another rel=canonical tag that points to the base URL. And vice versa: When the home page is set to "Tags", the URL /tags should not be indexed...

@franzliedke franzliedke changed the title NoJs Page SEO issues Prevent duplicate content for alternative home page URL Mar 13, 2017
@ardacebi

This comment has been minimized.

@luceos

This comment has been minimized.

@ardacebi

This comment has been minimized.

@franzliedke
Copy link
Contributor

The screenshot by @TurboProgramming are about a different problem... see issue #1140, which I just closed. 😁

(Moderator note: I am hiding the post with the screenshot - and some replies - so that we can focus on topic here.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants