Skip to content
New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Recent Activity improvements #71

Closed
mlandrum22 opened this issue Oct 24, 2019 · 21 comments
Closed

Recent Activity improvements #71

mlandrum22 opened this issue Oct 24, 2019 · 21 comments

Comments

@mlandrum22
Copy link

Our league differentiates (for keeper purposes) whether someone picks someone up via Waivers vs. via Free agency.

Would it be possible to add the difference when looking at recent activity?

Similar to only seeing trades, can there be a setting to only see "added" or "added by waivers"?

And finally, we are moving to FAAB next year (and would like to tie keeper values to FAAB spent), would it be possible to add in the monetary value of the waiver addition?

Thanks for an awesome API!

@cwendt94
Copy link
Owner

1st Question: Yes, I believe ESPN differentiates Waivers/Free Agency and I can add that to Recent Activity filter.

2nd Question: I am not very familiar with FAAB but I believe we can add the monetary value for successful wavier additions. However if you want all of the bids that were place even if they failed that data is located via another endpoint. Check out this PR #59 for more info.

@mlandrum22
Copy link
Author

Ohh that’s awesome news! Yeah I would only need successful bids.

@cwendt94
Copy link
Owner

Perfect. I will try to create a PR in the beginning of next week

@joshbis
Copy link

joshbis commented Oct 25, 2019

While you're at it, is it easy-ish to grab the dates of the transactions? I was curious to get time trends in trades, etc.

@mlandrum22
Copy link
Author

Oh, that would be a sweet add. We were thinking about tying keeper value to when they were picked up.

@cwendt94
Copy link
Owner

Yeah adding the dates should be really simple

@cwendt94
Copy link
Owner

Because I am not in a FAAB I am not able to see what field would be added for the bid in recent activity. If I could see a sample from someone in a FAAB league I can add the field.

I am almost done with filter and date for recent activity

@joshbis
Copy link

joshbis commented Nov 18, 2019

Were dates ever added into recent activity? If it turns out not to be simple, I can always just add them in manually.

@cwendt94
Copy link
Owner

Yes the dates have been added to recent activity

@joshbis
Copy link

joshbis commented Nov 18, 2019

awesome, must've been a version behind! thanks again.

@cwendt94
Copy link
Owner

No problem, and the wiki is up to date with the change!

@joshbis
Copy link

joshbis commented Nov 18, 2019

so the date is stored separately from the activity tuple?

@cwendt94
Copy link
Owner

Yeah, one date per activity

@joshbis
Copy link

joshbis commented Nov 18, 2019

cool - I just don't see it returned by:
league.recent_activity(1)

@cwendt94
Copy link
Owner

It was not added to the repr so it won’t show. You can always check https://github.com/cwendt94/ff-espn-api/wiki for what variables are returned and in each class

@joshbis
Copy link

joshbis commented Nov 18, 2019

ok - I don't see how it gets returned, but I am almost certainly missing something obvious.

@cwendt94
Copy link
Owner

So you should be able to do league.recent_activity(1).date

@joshbis
Copy link

joshbis commented Nov 18, 2019

hmm, when I do that I get:

league.recent_activity(1).date
---------------------------------------------------------------------------
AttributeError                            Traceback (most recent call last)
<ipython-input-6-1a439487e0b0> in <module>
----> 1 league.recent_activity(1).date

AttributeError: 'list' object has no attribute 'date'

@cwendt94
Copy link
Owner

Sorry meant to say league.recent_activity()[0].date because it returns a list of activities and each activity has a date

@joshbis
Copy link

joshbis commented Nov 18, 2019

got it!

@cwendt94
Copy link
Owner

cwendt94 commented Jan 5, 2021

You can now get bid_amount for wavier adds in the Recent Activity call. This is in v0.10.0

@cwendt94 cwendt94 closed this as completed Jan 5, 2021
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

No branches or pull requests

3 participants