Docstring is out of date? #481

New issue

Open

opened 2018-01-21 15:45:12 -08:00 by scottpham · 1 comment

scottpham commented

2018-01-21 15:45:12 -08:00

(Migrated from github.com)

It looks like the docstring on many functions are out of date. Namely, the URL references to Twitter documentation are incorrect.

For instance, the docstring for Twython.search returns:

search(self, **params)
    Returns a collection of relevant Tweets matching a specified query.

    Docs: https://dev.twitter.com/docs/api/1.1/get/search/tweets

The url given 404s. In this case I think it should be https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-tweets.html

It looks like the docstring on many functions are out of date. Namely, the URL references to Twitter documentation are incorrect. For instance, the docstring for `Twython.search` returns: ``` search(self, **params) Returns a collection of relevant Tweets matching a specified query. Docs: https://dev.twitter.com/docs/api/1.1/get/search/tweets ``` The url given 404s. In this case I think it should be https://developer.twitter.com/en/docs/tweets/search/api-reference/get-search-tweets.html

michaelhelmick commented

2018-01-22 06:57:37 -08:00

(Migrated from github.com)

Hi @scottpham, thanks for pointing this out. The docs point to their old documentation. Feel free to open up a pull request replacing all the references to dev.twitter.com

No labels

Bug

Enhancement

Feature Suggestion

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference: code/twython#481

No description provided.

Rows
Columns