Skip to content
This repository has been archived by the owner on Jan 6, 2024. It is now read-only.

Transactions: Post #39

Open
peacing opened this issue Jul 20, 2015 · 1 comment
Open

Transactions: Post #39

peacing opened this issue Jul 20, 2015 · 1 comment
Assignees
Milestone

Comments

@peacing
Copy link
Collaborator

peacing commented Jul 20, 2015

Add player
Drop Player
Add & Drop Player
Add player on Waivers
Add player on Waivers with FAAB (Free agent acquisition budget) bid
Propose Trades

@peacing peacing self-assigned this Jul 20, 2015
@peacing peacing added this to the 0.1.0 milestone Jul 20, 2015
josuebrunel referenced this issue in peacing/yahoo-fantasy-sport Jul 21, 2015
@josuebrunel
Copy link
Owner

Paul, i do believe that transaction doesn't support put but post. The method isn't that different anyway

def _post(uri, transaction):
       headers = {'Content-Type':'application/xml'}
       data = transaction.to_xml() 
       response = self.oauth.session.put(uri, data=data, headers=headers)
       return response

By the way, you haven't added the

return response

in the _put method, tests will definitely fail without that statement 😄

peacing added a commit to peacing/yahoo-fantasy-sport that referenced this issue Aug 2, 2015
peacing added a commit to peacing/yahoo-fantasy-sport that referenced this issue Aug 4, 2015
…ith multiple players), then will push to integrate branch
peacing added a commit to peacing/yahoo-fantasy-sport that referenced this issue Aug 7, 2015
…aiver and reject_trade to work, but still an error
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

2 participants