AndyReifman / ArseneWenger

1 stars 12 forks source link

Update Euro's command to only show relevant matches. #13

Closed AndyReifman closed 3 years ago

AndyReifman commented 3 years ago

Currently in the euro's command when I make a call to the library it returns a dictionary of all the matches in the competition.

I want the table output to only be about 5 matches but need to come up with an easy way to grab relevant ones, otherwise it will only show the initial 5 matches.

My thought for a quick and easy fix is creating a datestring for the current date and passing that to my call to get all matches on the current day. Then I will loop through the days until I get 5 matches or hit the final on the 11th of July.

AndyReifman commented 3 years ago

Added a pretty simple while loop with an increasing date. Works for now. Will need to add more code to deal with the end of the tournament (assuming I want to continue having a table of size 5) but that's a problem for a month from now me.