Closed ghost closed 2 years ago
Added PR.
Nice, thanks for the PR!
Thanks for filing this PR, @BWPanda; I encountered these issues myself! Since there hasn't been a new release of this module I'm going to try to patch my site from your PR.
To be a bit more clear, in the module's current state, the og:image
property doesn't get read by Facebook Open Graph at all.
@jackaponte This PR has been merged and should be included in the latest release. Are you still having problems after updating to latest stable?
Ah, whoops, thanks for that heads up Jen--somehow I missed that a new stable release was out! Updating fixed that problem.
This caused regressions in a handful of the other metatags, since it changed all of them from name
to property
. I expect only some of them need to be changed, so revering that change to re-evaluate.
The regression gets fixed in this follow-up. This issue here could get closed by someone with sufficient permissions.
This module doesn't output the correct code for Open Graph Protocol standard metatags. Namely it should output:
meta
tag (e.g.<meta property="og:title" content="The Rock" />
(it currently outputs the 'name' attribute instead))html
tag (e.g.<html prefix="og: http://ogp.me/ns#">
)These examples come from: http://ogp.me/