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

Allows setting of the Source Title and Source URL #85

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

Conversation

Linus12
Copy link

@Linus12 Linus12 commented Jan 13, 2016

Added the setter for the source_title and source_url as well as adding them to the the detail() map.
This allows posters to set these values when posting new posts. While these values are not required,
Tumblr does encourage them when posting items that are not your own creations ;)

The VideoPost class is missing the definition and return of the field video_url. This adds this field as a valid return for this class.
Tumblr returns the total number of Draft Posts and the total number Queued posts for a blog, but they were not accessible through Jumblr. This requires an app to step through all the Drafts and Queued posts to "count" them; very much a waste of resources. Since Tumblr returns these values already, these changes allow the application to access the counts without having to "count them".
While the Tumblr interface returns the Total Number of Draft Posts and the total number of Queued Posts for a blog, the Jumblr interface did not include them. Thus calling applications were forced to "count" them directly instead; a resource wasting process. This change returns these two counts in the Blog class.
Updated testReaders to look for new draft and queued counts
Added the setter for the source_title and source_url as well as adding them to the the detail() map.
This allows posters to set these values when posting new posts. While these values are not required,
Tumblr does encourage them when posting items that are not your own creations ;)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants