friedrich-de / TMW_Bot

GNU General Public License v3.0
2 stars 5 forks source link

Media type barchart returns points when it should return ammount logged #16

Closed J-O-S-H-L closed 2 weeks ago

J-O-S-H-L commented 2 weeks ago

In this line of code: https://github.com/friedrich-de/TMW_Bot/blob/b49104308dbf6fc976ff2889083a37e18e547c93/cogs/immersion_stats.py#L75 a vestigular function input was not cleaned up properly (by me). This mis-match of positional arguments means that immerison_type is always using the default value None. This makes the process_bar_data function follow the else path and show points_receivedinstead of the desired ammount_logged.

Example

In this example image: image the graph should show 2, 3, 3 episodes logged, but the points are showing

friedrich-de commented 2 weeks ago

Looks good, good work!