CE should pass metatags.

CommentFileSizeAuthor
#6 ce_metatags-3072504-6.patch4.55 KBpagach
#3 ce_metatags-3072504-3.patch3.63 KBpagach
#2 ce_metatags-3072504-1.patch2.91 KBpagach

Comments

pagach created an issue. See original summary.

pagach’s picture

pagach commented
StatusFileSize
new ce_metatags-3072504-1.patch2.91 KB

Attached is a file that adds meta tags.

pagach’s picture

pagach commented
StatusFileSize
new ce_metatags-3072504-3.patch3.63 KB

Added ability to blacklist tags via settings.php in format:

$settings['blacklisted_metatags']['meta'] = ['viewport', 'msapplication-config'];
$settings['blacklisted_metatags']['link'] = ['apple-touch-icon'];
pagach’s picture

pagach commented
Status: Active » Needs review
fago’s picture

fago
Primary language German
Location Vienna
commented
Title: CE renderer pass metatags » Provide metatags as part of the CE rendered response
Status: Needs review » Needs work

ok, looks good. Let's move things into a separate method (generatedMetatags()) to keep the main method short and good to go then.

pagach’s picture

pagach commented
StatusFileSize
new ce_metatags-3072504-6.patch4.55 KB

Refactored to use separate method.

  • pagach committed e3e7c61 on 8.x-1.x 
    Issue #3072504 by pagach: Provide metatags as part of the CE rendered...
pagach’s picture

pagach commented
Status: Needs work » Fixed

Status: Fixed » Closed (fixed)

Automatically closed - issue fixed for 2 weeks with no activity.