skekre98 / NBA-Search

flask application designed to explore NBA statistics :basketball:
78 stars 75 forks source link

Add NBA Standings section in predictions web page #21

Closed skekre98 closed 4 years ago

skekre98 commented 4 years ago

Description of feature/enhancement Add a component to show NBA standings for eastern and western conference.

Description of implementation Add component under the NBA Playoffs block. Can keep the data in the standings static for now. Only requires update to /templates/predictions.html.

Why this is a good feature/enhancement Would be nice to have a component to show NBA standings. This will later be updated to a more automated process through web scraping like playoffs bracket is now.

Additional context n/a

MohammedBajuaifer commented 4 years ago

Hi, @skekre98 I would like to work on this issue if that's good.

Changes should be made to the frontend and the data is static for now, right?

skekre98 commented 4 years ago

Yes, please create separate standings for east and west under playoffs. Can you keep it in the format of team name, wins, and losses. I will go ahead and assign the issue to you, thanks 😄.

MohammedBajuaifer commented 4 years ago

Hey @skekre98, I've started working on this feature but I would like your comments regarding the design/structure of it. Here is a sample of what I did so far!

image

skekre98 commented 4 years ago

@MohammedBajuaifer this looks awesome! Could you make the background of the standings white like the playoff bracket? Besides that this is perfect. I'm guessing they are both tables so we can add as many rows as we'd like right?

MohammedBajuaifer commented 4 years ago

Hey @skekre98, Sure thing! Yes we can add as many rows as we want, was thinking we could make it dynamic once we scrape the standings

skekre98 commented 4 years ago

Awesome, I'll open an issue for automating the standings data once your changes are in!

MohammedBajuaifer commented 4 years ago

Here! I have opened a PR

image

skekre98 commented 4 years ago

Awesome! I will go ahead and close this issue, and we can discuss automation of this workflow in #38 if you'd like 😃