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

Can we remove canonical url? #703

Closed
anhld opened this issue Sep 5, 2015 · 9 comments
Closed

Can we remove canonical url? #703

anhld opened this issue Sep 5, 2015 · 9 comments

Comments

@anhld
Copy link
Contributor

anhld commented Sep 5, 2015

Is the canonical url @line 17 in client/views/main.html required to run Rocket.Chat? If not, can we remove it?

@geekgonecrazy
Copy link
Member

Not required. What's your thinking behind removal?

@anhld
Copy link
Contributor Author

anhld commented Sep 5, 2015

For SEO purpose. Also, can we make the site title, apple-mobile-web-app-title and application-name be configurable?

@geekgonecrazy
Copy link
Member

So removed versus updated? I don't know much about its effects on SEO. Any info you can provide there on that would be useful.

Also title is currently configurable its under administration->general->Site Name.

Unfortunately.. the way we are having to update it now is after it is delivered. We don't currently have a way to update the markup before delivering it. Seems to be a limitation of the template framework. It can't update anything in <head> so it might still effect SEO some.

But this is planned

@anhld
Copy link
Contributor Author

anhld commented Sep 6, 2015

Search engines index the url in canonical tag instead of the url on the address bar. For example: i set up Rocket.Chat on mychatdomain.com, but if i have demo.rocket.chat in the canonical tag, Google will index demo.rocket.chat instead of mychatdomain.com.

I know the site title is configurable in the admin, but, as you mentioned, it's set by Javascript on the client side and wouldn't be seen by search engine bots. Is it possible to do it by using settings from a JSON on running meteor build?

@geekgonecrazy
Copy link
Member

That makes sense. So removing it'll cause it to just index the current url?

Yeah so that's what we have to figure out. Ideally the solution would be as easy as possible for someone to change in all deployment situations. Not just meteor build since not everyone will deploy this way.

@anhld
Copy link
Contributor Author

anhld commented Sep 6, 2015

Yup, that's correct.

I see, i will try to find a way to do it then :)

@mitar
Copy link
Contributor

mitar commented Sep 7, 2015

You could just use Meteor.absoluteUrl() as a value for it?

@geekgonecrazy
Copy link
Member

Due to the way meteor renders that wouldn't be updated until after the page had reached the client and the javascript was executed. Which would defeat the purpose cause search bots would possibly read this before executing javascript. Seems its better not to have it.

@mitar
Copy link
Contributor

mitar commented Sep 7, 2015

Ah, true.

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

No branches or pull requests

3 participants