[balls] A simple int arg for health and attack bonus in /count and /admin count

Ballsdex-Team / BallsDex-DiscordBot

Collect and exchange countryballs on Discord

Other

95 stars 157 forks source link

[balls] A simple int arg for health and attack bonus in /count and /admin count #461

Open imtherealF1 opened 1 week ago

imtherealF1 commented 1 week ago

Describe the feature you'd like You would be able to type a number you want for the attack bonus or health bonus, and it would count the balls you have with that bonus only.

Additional context It would need checks to prevent numbers more than the settings.max_attack and settings.max_health.

imtherealF1 commented 3 days ago

Thinking about it again, I don't think the arguments will be necessary for /admin count. I already made the code for /balls count ready. and I want to hear opinions on whether I should also make it for /admin count or not.

flaree commented 17 hours ago

Personally i'm not a fan, I just think its more clutter for the arguments

imtherealF1 commented 14 hours ago

I think that /count should have some sorting options, and it only has 3 which isn't cluttered at all, and with the shinies being moved to special soon it will become 2, so I think that it wouldn't be a big issue.