Canonical on cms pages

[wakabayashi]

Hello

When I go on a cms page I havethe following canonical tag in the head: <link rel="canonical" href="https://www.spielezar.ch/index.php?controller=cms"> On the live demo store, it's the same. What is this about? Looks like a bug to me.

[yaniv14]

By looking at getSeoFields() in FrontController I can see we are missing a case for 'cms' and 'cms_category'. So it actually goes to default, and thats why you have a wrong canonical link.

[yaniv14]

I can do a PR's with a fix, but I am not sure whats the correct way. Should we remove canonical tag from cms pages?, whats the point of having a canonical tag of page with no similar content page? Maybe Lesley can answer to that. Is it a bad practice to have canonical tag pointing to the same page?

[lesley]

I would leave it and fix it. Its always good practice to have a canonical url on a page, most will actually point to themselves as well.

[yaniv14]

https://github.com/thirtybees/thirtybees/pull/344/commits/ae627413a16f41c3667658eb934b171e86647297