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

Add some missing interfaces; alter handling of missing public_key #16

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

Conversation

strix-technica
Copy link

Add three endpoints missing from interfaces.json added (posts.update, forums.addModerator and forums.removeModerator);

Somewhat cosmetic change: Some API calls (notably posts.create with certain args like ip_address) will not work as expected if both public_key and secret_key are passed, so alter the check in disqusapi.DisqusAPI.__init__() so that it doesn't complain when only secret key is provided.

David King added 4 commits August 10, 2014 16:37
Some API calls (notably posts.create with certain args like
ip_address) will not work as expected if both public_key and
secret_key are passed, so don't complain if only secret key
is provided.
@mattrobenolt
Copy link
Collaborator

@strix-technica Seems that you've broken tests. :)

@strix-technica
Copy link
Author

Well, that was embarrassing. I've fixed it, tested it with Travis on my own fork, and it appears to be good. (And if I read this right, that fix automatically got included in the pull request and Travis has already run and passed it).

@mattrobenolt mattrobenolt force-pushed the master branch 2 times, most recently from 0ba26c4 to ae338e0 Compare September 29, 2014 10:55
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.

2 participants