Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

More meta tags #28

Merged
merged 10 commits into from
Feb 26, 2013
Merged

Conversation

gnunicorn
Copy link
Member

Implements and fixes #27 . Adding Meta-Tags for OpenGraph, Twitter and Google+ to make it easier share- and findable.

// cc @stefanhoth , @stevenfarlie

@gnunicorn
Copy link
Member Author

Oh. And Facebook-Admins. Then we should be able to get insights on likes and stuff for articles, too I think.

@stefanhoth
Copy link
Contributor

How about placing a hashmap (dictionary?) in the config for author name (or ots-account name)-> gplus-account mapping? This way the pages could just have the author name and we can add more stuff to that later on.

@gnunicorn
Copy link
Member Author

@stefanhoth that sounds like a great idea. But I am not sure that is really possible with liquid-template-engine. I'll try to figure it out and come back with you.

@gnunicorn
Copy link
Member Author

Ok. Messed up with the branch, but all should be back in order now. We are now using a pre-defined "authors" dictionary from the config. Much more smoother, thanks for the idea, @stefanhoth . Pls review again.

stefanhoth added a commit that referenced this pull request Feb 26, 2013
This version looks very clear to me. Merging it.
@stefanhoth stefanhoth merged commit 8b68c7e into OpenTechSchool:gh-pages Feb 26, 2013
@gnunicorn gnunicorn deleted the more-meta-tags branch March 13, 2013 12:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement metatags
2 participants