Skip to content
This repository has been archived by the owner on Nov 30, 2017. It is now read-only.

Retrieve all lists from an account #25

Closed
wants to merge 1 commit into from
Closed

Retrieve all lists from an account #25

wants to merge 1 commit into from

Conversation

jverdeyen
Copy link

No description provided.

@jverdeyen jverdeyen mentioned this pull request Feb 11, 2014
@miguel250
Copy link
Owner

@jverdeyen is there any reason you created a new class file for list?

@jverdeyen
Copy link
Author

@miguel250 Looks the "cleanest", no?

@miguel250
Copy link
Owner

I think it will be confusing since now we have getList and getLists. Maybe the bundle should get rewritten to support version 2 of the API.

@jverdeyen
Copy link
Author

Okey, sounds like a good idea. Currently I'm using my feature branch in my composer file, so no need to merge if you don't like it.

@OskarStark
Copy link
Contributor

i think this is an lightweight and useful commit of @jverdeyen

we have to write a good documentation of each call

@miguel250
Copy link
Owner

@OskarStark this call is part of v3 of the API so I don't want to add without supporting all of the new methods in v3. I am planning to add support to v3 but I don't have that much time right now. I am open for a pull request that keeps v2 and v3 separate from each other.

@OskarStark
Copy link
Contributor

sorry my fault.
you are right @miguel250 we should than not implement this in this branch...

@miguel250 miguel250 mentioned this pull request Oct 7, 2014
24 tasks
This pull request was closed.
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.

3 participants