Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Next inventory list request wasn't passing proper values into query. #10

Merged
merged 2 commits into from Jun 8, 2012

Conversation

Projects
None yet
2 participants
Contributor

csaunders commented Jun 8, 2012

This means that accounts with lots of merchandise wouldn't fetch
correctly.

@jduff please review

csaunders added some commits Jun 8, 2012

Next inventory list request wasn't passing proper values into query.
This means that accounts with lots of merchandise wouldn't fetch
correctly.
Improved inventory multipage test to ensure that ssl_post
is being called with correct parameters.
Contributor

jduff commented Jun 8, 2012

looks good, :shipit:

csaunders pushed a commit that referenced this pull request Jun 8, 2012

Merge pull request #10 from csaunders/bugfix/multiple_page_inventory_fix
Next inventory list request wasn't passing proper values into query.

@csaunders csaunders merged commit 2200cb8 into Shopify:master Jun 8, 2012

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment