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

Set lineup team_id None #653

Open
rohanday3 opened this issue Nov 11, 2023 · 5 comments
Open

Set lineup team_id None #653

rohanday3 opened this issue Nov 11, 2023 · 5 comments

Comments

@rohanday3
Copy link

Make transfers finds the team_id but set_lineup doesnt make changes and shows team_id as None. Should be an easy fix.
Version 1.7.5

@jack89roberts
Copy link
Contributor

Could you copy in the whole error message, if you're getting one? And does it make a difference if you explicitly set the team ID (airsenal_set_lineup --fpl_team_id <your_team_id>)? It's working for me so unsure what's going on.

@rohanday3
Copy link
Author

rohanday3 commented Nov 12, 2023

There is no error message but it's definitely a bug as the team_id is present for make transfers but not for set lineup. If I set it explicitly it does work.

(airsenalenv) rohan@MobyDick:~$ airsenal_make_transfers Login attempt 1/3... Failed
Login attempt 2/3... Logged in successfully
Getting starting squad for 810358 <------------ SEE HERE

Transfers to apply for fpl_team_id: 810358 for gameweek: 13
 ===========================================================

Bank Balance Before transfers is: £0.5
+--------+-----------------------+-------+
| Status |          Name         | Price |
+--------+-----------------------+-------+
|  OUT   | Alisson Ramses Becker |  £5.5 |
|   IN   |     Sam Johnstone     |  £4.6 |
+--------+-----------------------+-------+
Bank Balance After transfers is: £1.4


Apply Transfers? There is no turning back! (yes/no)no
Not applying transfers.  Can still choose starting 11 and captain.
fpl_team_id is None <-------------------- SEE HERE

Login attempt 1/3... Failed
Login attempt 2/3... Logged in successfully

=== starting 11 ===


== GK ==

David Raya Martin (ARS)

== DEF ==

Pedro Porro (TOT)
Vladimír Coufal (WHU)
William Saliba (ARS)

== MID ==

Mohamed Salah (LIV)
James Ward-Prowse (WHU)
Jarrod Bowen (WHU)(VC)
Hwang Hee-chan (WOL)

== FWD ==

Erling Haaland (MCI)(C)
Julián Álvarez (MCI)
Yoane Wissa (BRE)

=== subs ===

Alexis Mac Allister (LIV)
Alisson Ramses Becker (LIV)
Pau Torres (AVL)
Gabriel dos Santos Magalhães (ARS)

Apply changes to lineup? (yes/no)

@jack89roberts
Copy link
Contributor

Does the lineup get applied correctly if you say yes? I think it's only a bug in what's printed, as it looks like that line is printed before we find your default team ID if one hasn't been passed in explicitly.

@jack89roberts
Copy link
Contributor

Or you can try pulling and installing AIrsenal from the latest develop branch, as I've just pushed a fix for the printed message.

@rohanday3
Copy link
Author

I will try again next week to make sure but the lineup was not applied correctly this week.

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

2 participants