search

dblock / slack-gamebot

Ping pong, chess, pool and other games bot for Slack.
MIT License
129 stars 103 forks source link

Change Default Leaderboard Max #167

Closed JoshHull closed 6 years ago

JoshHull commented 6 years ago

We were constantly typing in 'leaderboard 15' to see everyone on the list. Making the default larger made it feel more natural. If you want to see the top three or five or ten, you'd have to say that. Past 15, we don't really care too much so 15 seemed like a good default.

dblock commented 6 years ago

@JoshHull care to implement a per-team set so we can have that as a feature for others?

JoshHull commented 6 years ago

Oh, I see where you're going with that. I'll take a crack at it, and see if I can come up with something reasonable.

dblock commented 6 years ago

I'm going to close this PR since I won't merge a default change, but see https://github.com/dblock/slack-gamebot/issues/163 for the feature request. There's also someone else's code that gets pretty far into this.

dblock commented 5 years ago

This was implemented as a setting in https://github.com/dblock/slack-gamebot/commit/1cb826da8607661be5a554dd1ea832ad33898762.