Skip to content
This repository has been archived by the owner on May 30, 2020. It is now read-only.

Various fixes #3

Open
wants to merge 3 commits into
base: master
Choose a base branch
from
Open

Various fixes #3

wants to merge 3 commits into from

Conversation

opoku
Copy link

@opoku opoku commented Jul 8, 2016

  1. Moved __data__ and useragent attributes into the instance scope so that they aren't shared by all instances of OpenGraph
  2. Fix warning thrown by BeautifulSoup by providing appropriate initialization parameter
  3. Add a property to return the __data__ dictionary.

@jaywink
Copy link

jaywink commented Oct 14, 2017

@opoku I included these (except the property, not sure why it's needed?) and some other tweaks and fixes to my fork, if interested. It will be maintained as I'm using it actively for a project. It's Python 3 only though: https://github.com/jaywink/python-opengraph/blob/master/CHANGELOG.md

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants