Skip to content

WIP: Updating fork#6

Open
thomas-tharp wants to merge 38 commits intomasterfrom
updating-fork
Open

WIP: Updating fork#6
thomas-tharp wants to merge 38 commits intomasterfrom
updating-fork

Conversation

@thomas-tharp
Copy link
Copy Markdown

No description provided.

dghubble and others added 30 commits May 17, 2018 10:14
* Drop Go 1.6 and 1.7 coverage
* Fix DirectMessageEvent id and sender/recipient id to be
strings. Comparing API docs with prior docs shows the intended
shift toward strings
* Deprecate DirectMessage service's Show, Get, New, Sent,
and Destroy methods.
* Add small example of usage
* Apps string field doesn't match actual JSON responses
* golint previously supported both a golang.org and
github.com import path. Recently, they switched to
allowing only the golang.org path
* golang/lint@9a27203
* Test stream server HandleFunc's stream an expected
message and then return an HTTP error response code
to break stream retrying, so received messages can be
checked
* Tests erroneously used 130 as the HTTP error response
code (130 is the Twitter code). Starting in Go v1.11,
such invalid HTTP response codes cause the connection to
the test stream server to hang, causing tests to hang.
* Fix the error code to be 500, some valid HTTP status
code that isn't handled specially (e.g. no backoff)
* Add `EventsNew` method for sending a Direct Message event
* Add `EventsShow` method for getting a single Direct Message event
* Add missing test coverage for `EventsList` method
* Add`EventsDestroy` method for destroying a Direct Message event
* Move DirectMessageEvents example to its own `go run`-able file
* https://developer.twitter.com/en/docs/direct-messages/api-features
* Roll out standard Makefile across projects
* user_id is a comma separated list of user IDs
* Add `Since` and `Filter` to SearchTweetParams
* Ensure examples and their dependencies aren't included in
the twitter package dependencies
@thomas-tharp thomas-tharp changed the title Updating fork WIP: Updating fork Jul 23, 2019
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.