Twitter Card metadata #813

Closed
barnabywalters opened this Issue Jul 10, 2012 · 8 comments

Comments

Projects
None yet
2 participants
Contributor

barnabywalters commented Jul 10, 2012

Could you add twitter card photo meta tags to photo pages, please?

Owner

jmathai commented Jul 10, 2012

Have a look at the Facebook plugin which inserts open graph tags.

https://github.com/openphoto/frontend/blob/master/src/libraries/plugins/FacebookLikePlugin.php

Contributor

barnabywalters commented Jul 10, 2012

Ah, brilliant! Didn't know OpenPhoto supported plugins.

Owner

jmathai commented Jul 10, 2012

Pull request :).

We don't have a good interface to add/remove plugins but it's in the site config typically found at src/userdata/configs/{hostname}.ini.

Contributor

barnabywalters commented Jul 11, 2012

Great. I'm away in france until mon but will sort it out then.

Contributor

barnabywalters commented Aug 3, 2012

Right, trying to experiment with plugins but I can't figure out what format the config file wants the list in. Is it a JSON array of paths to the plugin files? Comma separated list?

Owner

jmathai commented Aug 3, 2012

It's ini style. I need to add the interface back in for plugins since the app itself manages config values for the plugin. For testing purposes you can set default values in the defineConf.

  public function defineConf()
  {
    return array(
        'key1' => 'val1',
        'key2' => 'val2'
    );
  }

The UI would generate the proper config and then the default values would be overridden. But for now don't worry about the config files :).

Contributor

barnabywalters commented Aug 3, 2012

Ah, sorry, I meant "how do I get the list of active plugins to work"? E.G. I have a plugin but can't figure out how to get OP to start using it!

Owner

jmathai commented Aug 3, 2012

Ah, comma separated list. Leave off the "Plugin" like so...

[plugins]
activePlugins="FacebookConnect,GoogleAnalytics"

@jmathai jmathai closed this in 28e2480 Nov 28, 2012

jmathai added a commit that referenced this issue Nov 28, 2012

jmathai added a commit that referenced this issue Mar 14, 2013

jmathai added a commit that referenced this issue Mar 14, 2013

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment