Business API was added #95
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Added support for IAF Tokens in Trading API
- changed ebaybase:_execute_http_request() to trap exceptions - added ebaybase:_get_resp_body_errors() to handle XML body errors - cleanup
Added SOAP-style class and other changes
Handle cases when _get_resp_body_errors() may get called before execute()
Last change, I promise :-) Use "" as default for response_data
and moves it to _version.py. Loading __init__.py in setup.py was preventing the dependencies from being installed, since they are imported in __init__.py. This solution was found here: http://stackoverflow.com/a/7071358
Fix issue timotheus#3 which allows installation via setup.py
Update README.rst
Fix find item to allow alternate domain to be specified.
Return the result of the Finding API next_page method.
|
This is a great addition. Thank you. Would you be willing to add some tests for this? Either doc tests or something in the samples directory (https://github.com/timotheus/ebaysdk-python/tree/master/samples). It'll help ensure nothing breaks this API when future changes come in. -Tim |
Conflicts: ebaysdk/.DS_Store
|
Hi - I added this code to the master repo along with some samples. Unfortunately the MR was issued prior to several unicode fixes so I couldn't easily merge it. Instead I added the code and made a few changes. Thanks for committing this work and sorry I couldn't use your pull request. |
timotheus
force-pushed
the
master
branch
2 times, most recently
from
5439bd8
to
059cdd8
Compare
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Business API was added