forked from rDrama/rDrama
1
0
Fork 0
Commit Graph

13 Commits (4a05910161062504f6ce14cfd360732d1c351459)

Author SHA1 Message Date
justcool393 8bd637df57 leaderboards: use len 2022-11-01 03:04:56 -05:00
justcool393 08700a65b0 fix prev commit 2022-11-01 02:59:36 -05:00
justcool393 5c32713469 leaderboards: workaround jinja can't do this by setting a value for if v appears in ranking 2022-11-01 02:57:10 -05:00
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