Skip to content
This repository was archived by the owner on Jan 21, 2020. It is now read-only.

Conversation

@BnitoBzh
Copy link

@BnitoBzh BnitoBzh commented Nov 9, 2017

Example : https://api.twitter.com/1.1/statuses/update.json is called like this :
POST https://api.twitter.com/1.1/statuses/update.json?status=Maybe%20he%27ll%20finally%20find%20his%20keys.%20%23peterfalk

Body content is not used by the Twitter API and returns "Missing required parameter: status"

see documentation

@weierophinney
Copy link
Member

Please look at the Travis testing results; the changes you propose break existing functionality.

Also, please provide unit tests covering the changes you propose.

@weierophinney
Copy link
Member

Closed in favor of #53, which fixes this properly.

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