Skip to content
This repository has been archived by the owner on Sep 16, 2021. It is now read-only.

Change Canonical link default #167

Open
wouterj opened this issue May 11, 2014 · 9 comments
Open

Change Canonical link default #167

wouterj opened this issue May 11, 2014 · 9 comments
Milestone

Comments

@wouterj
Copy link
Member

wouterj commented May 11, 2014

I recently watched this talk by Matt Cutts about the canonical link element: https://www.youtube.com/watch?v=Cm9onOGTgeM

Here, he states that the canonical link element should be the last thing to do when having duplicate content. Things you should do first are 301 redirects and a good sitemap.

We currently default to the canonical link usage. Shouldn't we change it to 301 redirect instead?

@ElectricMaxxx
Copy link
Member

👍 for change.

@dbu
Copy link
Member

dbu commented May 11, 2014

fine for me. but if we do that before announcing the cmf release, we should release a 1.0.1 version and note the BC break in the changelog. if we only do it after, it should go into 1.1 probably

@ElectricMaxxx
Copy link
Member

What to do here as we forgot that issue?

@dbu
Copy link
Member

dbu commented Jul 10, 2014

put it into 1.1 and do a changelog entry?

@wouterj
Copy link
Member Author

wouterj commented Jul 10, 2014

We should change the whole canonical link thing imo. Canonical links serve more purposes than replacing the redirect.

@dbu
Copy link
Member

dbu commented Sep 20, 2015

is this now a 2.0 issue or should we do it for 1.2? doing it in 1.2 could be unexpected.

@wouterj
Copy link
Member Author

wouterj commented Sep 20, 2015

Yeah, let's skip for 2.0

@wouterj wouterj modified the milestone: 2.0 Sep 20, 2015
@ElectricMaxxx
Copy link
Member

👍

Am 20.09.2015 um 15:03 schrieb Wouter J:

Yeah, let's skip for 2.0


Reply to this email directly or view it on GitHub:
#167 (comment)

@wouterj
Copy link
Member Author

wouterj commented Feb 10, 2016

What about trigger a deprecation notice when the configuration value is not set (so the default of canonical is used) to warn users that this behaviour will change in 2.0? This means people are aware of explicitely setting this to canonical (in case they want to keep the current behaviour) or can have forward compatiblity by setting it to redirect.

@ElectricMaxxx ElectricMaxxx modified the milestones: 2.1, 2.0 Nov 28, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

3 participants