Conversation
|
I've done some local testing and it's all working now.
|
|
Checks are being a pain in the butt. Are you good to use your branch version til I can look at it properly? Anyone else looking at this, you can install with |
|
Yeah I have no problem using my branch as the ffscraper package locally. Don't sweat it |
|
Okay, reviewing a little more closely now that I'm back. ff_draftpicks is designed to return the following:
Sleeper used to make the former available but now only makes it available if the league is actively drafting - otherwise it only makes the traded picks available and (sometimes) a draft order. To recreate the previous functionality, I envision sleeper_draftpicks to need to do:
|
|
I'm trying to button up all these things this week before I get going on newjob so I should have some time to look at this today/tomorrow. |
3b5e8fc to
e28ab04
Compare
|
Going to accept this PR so that ff_draftpicks is not entirely broken and open a new issue (#386) for the optimal behaviour. |
Fixes #352
ff_draftpicksis returning an error when trying to load sleeper data. It seems the current_picks object was removed