PLAT-142 'Metatags' module global configurations for SEO and social media

[vireshpatel]

Ticket

https://jira.comicrelief.com/browse/PLAT-142

Changes proposed in this pull request

• Set Metatags global configurations (site-wide)
• Set content types to use global configurations
• Set Metatags module taxonomy configurations (og:title added to suppress php error being thrown)
• Set Metatags module user configurations (og:title added to suppress php error being thrown)
• Metatags module user configurations added to CR config profile

Notes:

• Image fields for og:image not included as feature isnt working - bug in module - raising separate ticket and action
• Meta description tag not included - to determine best of course of action later (discussed in SEO doc), summary field pulls in html markup (not best practice) - raising separate ticket and action
• CR SEO doc with links to resources and tools:
  https://docs.google.com/document/d/1pl30E0ZlLzPs_O-qM5e3admY-SZOzLeMC5x-eu_LbEE/edit
• Drupal core also outputs its own metatags - separate ticket being raised to suppress them
  -Behat full tests to be implemented in different ticket and PR (ones originally implemented in PR where partial so removed accordingly)

[bimsonz]

@pvhee looks good to me, thoughts?

[pvhee]

@vireshpatel don't think we need metatags for users since we don't have users on the site? should we remove this snippet?

[vireshpatel]

@pvhee by default there are metatag settings for users and taxonomy.

I had to add the yaml config in as a work around to stop both entity types from throwing errors (and logging them into the watchdog) and until a solution has been found - this is a reported issue: https://www.drupal.org/node/2699487

Cavet for removing config will log errors when people sign into the CMS or are already signed and visit the homepage.

Only other alternative is to swtich metatag off until and revisit at a later date.

[pvhee]

@vireshpatel ok makes sense! let's leave this in for now then and keep metatag turned on

[vireshpatel]

@pvhee Behat feature tests and feature context removed - to be reintroduced and refined in new ticket and pr.

Ready to merge in.

[pvhee]

@vireshpatel cool, merging in