-
Notifications
You must be signed in to change notification settings - Fork 99
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
apply_live_bonus #75
base: master
Are you sure you want to change the base?
apply_live_bonus #75
Conversation
This seems like a mix of other pull requests you have opened. For example I just merged #73. It's hard to tell what you are actually adding here that wasn't already added in other pull requests / has been merged already. |
It's dependant on previous commits.
The main change is in the get_gameweek method, where the bonus points are
retrieved and then assigned as attributes to the player
…On Sun 20 Oct 2019, 13:34 Amos Bastian ***@***.***> wrote:
This seems like a mix of other pull requests you have opened. For example
I just merged #73 <#73>. It's hard
to tell what you are actually adding here that wasn't already added in
other pull requests / has been merged already.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#75?email_source=notifications&email_token=AF7ZQIIRKNVGMQJE2DYTV2TQPRF65A5CNFSM4JCP2U4KYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOEBYJDSI#issuecomment-544248265>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AF7ZQIPCS2FIHI66EUMOCTDQPRF65ANCNFSM4JCP2U4A>
.
|
I think a lot of this has been merged in already (e.g. calculating the bonus points in the |
apply_live_bonus