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

Issue 65 - Add Twitter Cards metadata #66

Merged
merged 2 commits into from
Jan 5, 2017
Merged

Issue 65 - Add Twitter Cards metadata #66

merged 2 commits into from
Jan 5, 2017

Conversation

rodfersou
Copy link
Member

closes #65

@rodfersou rodfersou force-pushed the issue_65 branch 3 times, most recently from 9167985 to d0276d8 Compare May 31, 2016 17:52
@@ -35,6 +38,7 @@ def __init__(self, context, request):
self.url = context.absolute_url()
self.language = get_language(context)
self.sheet = getattr(pp, 'sc_social_likes_properties', None)
self.image = get_content_image(context, width=1200, height=630)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rodfersou why are you hardcoding this with those valued?

@rodfersou rodfersou force-pushed the issue_65 branch 5 times, most recently from faa0758 to 2934378 Compare June 9, 2016 21:17
@rodfersou rodfersou force-pushed the issue_65 branch 2 times, most recently from 5ae3901 to 5cac285 Compare December 21, 2016 13:42
@hvelarde hvelarde force-pushed the issue_65 branch 3 times, most recently from ad90dfb to 595cc9f Compare January 5, 2017 16:54
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.

Twitter Cards metadata is not being generated
2 participants