Commit Graph

10 Commits (40a34ea26a1f120a9c59e11a8d43be34e9839f4d)

Author SHA1 Message Date
justcool393 e90edce28a leaderboard: fix bug with hat leaderboard
in my defense, there was a lot of stuff
2022-11-01 02:21:14 -05:00
justcool393 9048f5127d leaderboards: fix usernames in tables i think
i'm not *happy* about this, something seems inelegant about this solution...
2022-11-01 02:17:33 -05:00
justcool393 55c8ce6e9d leaderboard: set value func in all cases 2022-11-01 01:36:41 -05:00
justcool393 02dde2a96f leaderboard: check for None explicitly instead of implicitly
nonetype is used in only one case, if the value is implicitly falsey (like a 0) it can *potentially* cause issues
i have no idea how to repro this, apparently @Aevann1 was able to and it's a issue on rdrama.net, but not devrama or my own testing environment
2022-11-01 01:28:18 -05:00
Aevann1 5719688178 use tabs, not spaces 2022-10-29 01:39:31 +02:00
justcool393 efd63b8666 leaderboards: show user as last row if not in lb 2022-10-28 12:36:18 -07:00
justcool393 fb727ac40c lambda fns 2022-10-28 12:36:18 -07:00
justcool393 cfeafea0d4 leaderboards: annotate the db param, fix posts and comments, and move const to const file, etc 2022-10-28 12:36:18 -07:00
justcool393 89e10db2e0 leaderboards: fix prev commit i think 2022-10-28 12:36:18 -07:00
justcool393 8873171f5c leaderboards: refactor leaderboards to their own class
right now, leaderboards are complex enough that they ought to be upgraded to at least a second class thing.
this commit provides an *okay* implementation of a per-request leaderboard
there are many things to be done, including caching, persistence, etc
i don't like this like 80 parameter __init__ but it's what i've got without overengineering it imo
this is potentially already overdoing it
2022-10-28 12:36:18 -07:00