I have several content types that are scheduled events. As things are now, I can only select article as the og type. According to http://developers.facebook.com/docs/reference/opengraph/object-type/event, this is what it should be listed as. I tried to hack the module myself, and I was able to add the event og type. Facebook didn't like it because the page also needs event:start_time. My node does have the start date time, but if I'm reading the Facebook error correctly, it want's it in php date_time format. Mine is in a readable format.
Do you think this could be possibly added to the module?
Comment | File | Size | Author |
---|---|---|---|
#23 | metatag-n2026343-23.patch | 19.19 KB | DamienMcKenna |
Comments
Comment #1
wxman CreditAttribution: wxman commentedI also found that article:publisher is becoming more important as of late. I hacked the module metatag_opengraph.metatag.inc adding this:
This seems to work fine.
Comment #2
DamienMcKennaLets focus this on identifying other OG tags & tag options that would be useful to have.
Comment #3
wxman CreditAttribution: wxman commentedIt seem like they have been changing things around a lot lately. I've been trying to keep up.
Comment #4
paulkittredge CreditAttribution: paulkittredge commentedAccording to https://developers.facebook.com/blog/post/2013/06/19/platform-updates--n..., Facebook now has article:publisher and article:author. Can those be added to the next release?
Thanks!
Comment #5
valkum CreditAttribution: valkum commentedHmm i think the best for a short result would be some hooks in _metatag_opengraph_type_options and metatag_opengraph_metatag_info
Then you can add more Modules, which will add specific og types etc.
Comment #6
valkum CreditAttribution: valkum commentedHere is some work done by me.
Added a module for opengraph events. But there ist a bug with event:fbid (https://developers.facebook.com/bugs/321976687938505?browse=external_tas...)
Patch needs a bit more work.
Check for token with date for event:start_time in format: e.g 2013-09-07T16:00:00+0200 or create one.
Maybe move event tags into new metatag group but its only a visual thing.
Comment #7
Simon Georges CreditAttribution: Simon Georges commentedCross-referencing #2051155: Add option for video/mp4 to Open Graph video types.
Comment #8
valkum CreditAttribution: valkum commentedComment #10
skruf CreditAttribution: skruf commentedHere's a patch to add the article publisher and author tags as specified here:
https://developers.facebook.com/blog/post/2013/06/19/platform-updates--n...
Comment #11
skruf CreditAttribution: skruf commentedComment #12
DamienMcKennaThere's a tiny typo: "Links and article" should be : "Links an article"
Comment #13
skruf CreditAttribution: skruf commentedUpdated with typo fix
Comment #14
DamienMcKennaAnother small typo - it should be "author's" and "publisher's".
Also, would you mind expanding this to cover the other article sub-tags?
Comment #15
DamienMcKennaClosed a duplicate: #2133123: Support for Article object type
Comment #16
skruf CreditAttribution: skruf commentedSure, I'll take this on.
Comment #17
DamienMcKennaOther tags to add:
Also, any meta tags that are marked as "array" need to be flagged as "multiple" in the API so that they can properly handle multiple values.
Comment #18
DamienMcKennaThis adds the following meta tags:
It also:
Comment #19
DamienMcKennaNeed to add another few tags from http://ogp.me.
Comment #20
DamienMcKennaAdding this to the 1.0 release plan.
Comment #21
DamienMcKennaThis also adds the following meta tags:
Comment #22
DamienMcKennaThis one also adds the following:
Comment #23
DamienMcKennaImproves the patch from #22.
Comment #24
DamienMcKennaCommitted.
Comment #26
DamienMcKennaFYI: #2319389: Add extra meta tags for og:type=video