Comments

DamienMcKenna created an issue. See original summary.

it-cru’s picture

Does this issue also includes meta tags like article:publisher, article:published_time and so on from http://ogp.me/ ?

damienmckenna’s picture

The idea is to add anything listed for the D7 version of Metatag that's not already in the D8 version, so probably.

it-cru’s picture

Patch for article: xyz meta tags ported from current metatag D7 1.14 module. Works fine on first test for me. Labels, descriptions and field weights are taken from the 7.x-1.14 version. Hope file, class and ID namings are correct for D8.

damienmckenna’s picture

Status: Active » Needs review

Status: Needs review » Needs work
damienmckenna’s picture

Status: Needs work » Needs review
StatusFileSize
new0 bytes

Updated to add the new 'secure' attribute.

Status: Needs review » Needs work

The last submitted patch, 10: metatag-n2692117-10.patch, failed testing.

The last submitted patch, 10: metatag-n2692117-10.patch, failed testing.

damienmckenna’s picture

Status: Needs work » Needs review
StatusFileSize
new8.53 KB

Doh!

  • DamienMcKenna committed a66eb05 on 8.x-1.x authored by IT-Cru
    Issue #2663974 by IT-Cru, DamienMcKenna: Added the og 'article' meta...
damienmckenna’s picture

Status: Needs review » Needs work

Committed. Putting this back to 'needs work' as there are more to go.

damienmckenna’s picture

it-cru’s picture

If the last patch was flawless (exclude the missing new 'secure' attribute), I can gladly prepare patches for the outstanding OG metatags.

damienmckenna’s picture

IT-Cru: Any help would be appreciated.

damienmckenna’s picture

Not putting this into beta10.

dawehner’s picture

Status: Needs work » Needs review
StatusFileSize
new5.21 KB

Here are some additional ones coming from #2799949: A couple of additional metatags

damienmckenna’s picture

StatusFileSize
new7.08 KB
new2.61 KB

Thanks @dawehner.

This adds the new tags to the module's tests suite, and adds the config schema records.

damienmckenna’s picture

Status: Needs review » Fixed
Related issues: +#2788195: Plan for Metatag 8.x-1.0-beta11 release

Committed. Lets do new issues for any additional OG meta tags.

dawehner’s picture

This adds the new tags to the module's tests suite, and adds the config schema records.

Nice, thank you for adding them! One thing I'm a bit confused about: Why does this needs to be a label, do you want to have different values per language?

damienmckenna’s picture

Yes, the goal is to allow each tag to be translated as there will be some sites that have different versions of the same video depending upon the language.

Status: Fixed » Closed (fixed)

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