seo-bundle icon indicating copy to clipboard operation
seo-bundle copied to clipboard

Change Canonical link default

Open wouterj opened this issue 11 years ago • 9 comments

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?

wouterj avatar May 11 '14 11:05 wouterj

:+1: for change.

ElectricMaxxx avatar May 11 '14 11:05 ElectricMaxxx

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

dbu avatar May 11 '14 13:05 dbu

What to do here as we forgot that issue?

ElectricMaxxx avatar Jul 09 '14 21:07 ElectricMaxxx

put it into 1.1 and do a changelog entry?

dbu avatar Jul 10 '14 06:07 dbu

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

wouterj avatar Jul 10 '14 06:07 wouterj

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

dbu avatar Sep 20 '15 11:09 dbu

Yeah, let's skip for 2.0

wouterj avatar Sep 20 '15 13:09 wouterj

:+1:

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: https://github.com/symfony-cmf/SeoBundle/issues/167#issuecomment-141784143

ElectricMaxxx avatar Sep 20 '15 18:09 ElectricMaxxx

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.

wouterj avatar Feb 10 '16 11:02 wouterj