Add lists/* functions to API

Pretty straightforward, map out all of the twitter list functions to functions in the API, found here: https://dev.twitter.com/docs/api/1.1#99

Did some initial work for things I needed, but I haven't mapped them all out as I had no use cases for testing to make sure everything was hooked up right. If someone else wants to continue off of my work, that'd be awesome.

Comment	File	Size	Author
#10	twitter-n2028245-10-7.x-6.x.patch	9.67 KB	damienmckenna
#6	twitter-n2028245-6.patch	10.18 KB	damienmckenna
#1	twitter-lists-api-2028245.patch	10.39 KB	wbobeirne

Comments

Comment #1

wbobeirne commented 26 June 2013 at 03:57

Status:

Active

» Needs work

Status	File	Size
new	twitter-lists-api-2028245.patch	10.39 KB

Attaching patch. Forgot to mention that this patch also adds some things that I needed for my use cases to the status object. Sorry for it being kind of out-of-place.

Log in or register to post comments

mfb’s picture

Comment #2

mfb

they or he

commented 28 June 2013 at 00:29

This is great, I was just working on a similar patch.

Shouldn't pluralization in a method name match the API endpoint? I.e. if the API is 'lists/statuses' then method would be lists_statuses()?

Log in or register to post comments

Comment #3

wbobeirne commented 2 July 2013 at 19:26

Glad it's helpful mfb. I leave that decision to someone with more involvement in this project, but I see what you mean.

Log in or register to post comments

damienmckenna’s picture

Comment #4

TN, USA

commented 19 May 2015 at 18:16

Issue summary:	View changes
Status:	Needs work	» Needs review

Log in or register to post comments

Comment #5

19 May 2015 at 18:46

Status:

Needs review

» Needs work

The last submitted patch, 1: twitter-lists-api-2028245.patch, failed testing.

Log in or register to post comments

damienmckenna’s picture

Comment #6

TN, USA

commented 20 May 2015 at 09:48

Status:

Needs work

» Needs review

Status	File	Size
new	twitter-n2028245-6.patch	10.18 KB

1 file was hidden/shown/deleted

Status	File	Size
hidden	twitter-lists-api-2028245.patch	10.39 KB

Rerolled.

Log in or register to post comments

damienmckenna’s picture

Comment #7

TN, USA

commented 22 August 2015 at 13:16

Status:

Needs review

» Needs work

Log in or register to post comments

Comment #8

22 August 2015 at 13:16

Status:

Needs work

» Needs review

DamienMcKenna queued 6: twitter-n2028245-6.patch for re-testing.

Log in or register to post comments

damienmckenna’s picture

Comment #9

TN, USA

commented 30 August 2015 at 09:06

Status:

Needs review

» Needs work

@wbobeirne: Did you forget to include part of the patch? There's no definition for the TwitterLists class but it's used in the get_lists() method.

Log in or register to post comments

damienmckenna’s picture

Comment #10

TN, USA

commented 30 August 2015 at 09:06

Status	File	Size
new	twitter-n2028245-10-7.x-6.x.patch	9.67 KB

Here's a 7.x-6.x version of the patch.

Log in or register to post comments

damienmckenna’s picture

Comment #11

TN, USA

commented 30 August 2015 at 09:06

Status:

Needs work

» Needs review

Log in or register to post comments

damienmckenna’s picture

Comment #12

TN, USA

commented 30 August 2015 at 09:07

Status:

Needs review

» Needs work

Log in or register to post comments

Comment #13

30 August 2015 at 09:07

The last submitted patch, 10: twitter-n2028245-10-7.x-6.x.patch, failed testing.

Log in or register to post comments

damienmckenna’s picture

Comment #14

TN, USA

commented 4 September 2015 at 09:07

Version:	7.x-5.x-dev	» 7.x-6.x-dev
Status:	Needs work	» Needs review

Log in or register to post comments

Comment #15

4 September 2015 at 09:07

DamienMcKenna queued 10: twitter-n2028245-10-7.x-6.x.patch for re-testing.

Log in or register to post comments