Skip to content

Conversation

jtallieu
Copy link
Contributor

I was working on this API for a friend, who later told me about this project after I was almost finished. After taking a look, I noticed that iterating over more than 250 resources and filtering were not supported. Please pull if you like where I am heading with this project. Support for updating and creating will be coming soon.

Thanks,

Support for pagination of resources, filtering and minimal update
support.
Added read_only fields to Products and Images.
Examples of product and image updates in a script.
Fixed the pagination to short circuit if the number of items in he page 
are less than the max, instead of waiting for an HTTP 204.
Added more examples and test code
@maetl
Copy link
Contributor

maetl commented Nov 15, 2012

This is cool, thanks for the pull request.

I’m currently going through the code and will point out a few small areas which aren’t quite what we’d expect.

@@ -17,21 +17,144 @@ Usage:

```
#!/usr/bin/python
import bigcommerce.api
from bigcommerce.api import bigCommerce
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is okay for your own applications, but as a public facing API client for Bigcommerce, we’d want to keep the naming consistent with our brand, which is now Bigcommerce across all media, not BigCommerce or bigCommerce (bigcommerce in all lowercase is acceptable in some contexts).

@jtallieu
Copy link
Contributor Author

OK, see if you like.

@maetl
Copy link
Contributor

maetl commented Dec 4, 2013

Hey, I didn’t realise this had been updated. Sorry for the confusion.

@maetl maetl mentioned this pull request Dec 5, 2013
@maetl
Copy link
Contributor

maetl commented Dec 11, 2013

👍

We’re going to merge this, and take a shot at improving it to support all the recent API changes and improvements, as well as fold in changes from #3.

maetl added a commit that referenced this pull request Dec 11, 2013
@maetl maetl merged commit ef39f19 into bigcommerce:master Dec 11, 2013
@jtallieu
Copy link
Contributor Author

Excellent. We recently added create and update support. To be released soon.

Sent from my iPhone

On Dec 10, 2013, at 11:38 PM, Mark Rickerby notifications@github.com wrote:

Merged #2.


Reply to this email directly or view it on GitHub.

maetl added a commit that referenced this pull request Feb 6, 2014
Lots of imports fixed. Create and delete implemented.
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.

2 participants