search

LinsaneFC / RasenBot

3 stars 0 forks source link

Made changes to the display of game info #34

Open benjaminl4917 opened 1 year ago

benjaminl4917 commented 1 year ago

Created a set that tracks the members online. Timed update now updates all members online at once

stephenlee91 commented 1 year ago

DId we confirm that this works for video games and not just spotify? Also it seems that this displays the time spent per day. What if this gets really big?

LinsaneFC commented 1 year ago

Did we confirm that this works for video games and not just spotify? Also it seems that this displays the time spent per day. What if this gets really big?

@benjaminl4917 I think the check has been made for video games right? @stephenlee91 What do you mean by if this gets really big? The database? Or...?

benjaminl4917 commented 1 year ago

I believe it should work for video games

stephenlee91 commented 1 year ago

Did we confirm that this works for video games and not just spotify? Also it seems that this displays the time spent per day. What if this gets really big?

@benjaminl4917 I think the check has been made for video games right? @stephenlee91 What do you mean by if this gets really big? The database? Or...?

The database. Specifically for one person. Let's say the database goes on for 1 year and a person listens to spotify daily. If the person types in $get_info it could be a really long page full of information.

LinsaneFC commented 1 year ago

Did we confirm that this works for video games and not just spotify? Also it seems that this displays the time spent per day. What if this gets really big?

@benjaminl4917 I think the check has been made for video games right? @stephenlee91 What do you mean by if this gets really big? The database? Or...?

The database. Specifically for one person. Let's say the database goes on for 1 year and a person listens to spotify daily. If the person types in $get_info it could be a really long page full of information.

I'm not as concerned with the output because we can always change that. Right now it's good to show all just to make sure day by day is logged in database. Future wise we can show weekly, have different commands to show different information (including all), or just simply show current day/week, or show total playtime as one output. Good idea to think about this in the future so we can have a low priority ticket for it since it's more of a quality of life thing. Backend should be the main focus for now.

LinsaneFC commented 1 year ago

Big new commit. 👍 Display info previously would call X times for X servers that each person was a part of.

IE: LinsaneFC part of Konoha and Testing Server 1 and Testing Server 2. The update code would be triggered 3 different times.