search

idkclub / dicebot

🎲 /roll support for Slack
https://dicebot.idk.club
The Unlicense
53 stars 20 forks source link

Compact Version or way to hide #47

Open SeaPlusPlush opened 5 years ago

SeaPlusPlush commented 5 years ago

Hello,

When my workspace uses dicebot for public stat rolling, it really clogs up our channel (see below image). While I love the breakout, it adds quite a lot of clutter and I can't figure out how to remove or hide the dicebot messages (maybe there are ways but FYI I'm not an admin).

Anyway, I was wondering if there were a way for rolls like this to be compacted.

For instance, maybe running the command from the pic below (with an additional keyword)...

/roll compact 4d6k3 for Stat1, 4d6k3 for Stat2, 4d6k3 for Stat3, 4d6k3 for Stat4, 4d6k3 for Stat45, 4d6k3 for Stat6

...has the following result:

@.Person rolled 12 for Stat1 +7 for Stat2 +13 for Stat3 +11 for Stat4 16 for Stat5 +14 for Stat6 = 73 (Dice: 4d6)(Keep: 3)(Rolls: 3 4 5)(Removed: 1) Total = 12 (Dice: 4d6)(Keep: 3)(Rolls: 1 1 5)(Removed: 1) Total = 7 (Dice: 4d6)(Keep: 3)(Rolls: 4 4 5)(Removed: 3) Total = 13 (Dice: 4d6)(Keep: 3)(Rolls: 2 3 6)(Removed: 2) Total = 11 (Dice: 4d6)(Keep: 3)(Rolls: 4 6 6)(Removed: 2) Total = 16 (Dice: 4d6)(Keep: 3)(Rolls: 2 6 6)(Removed: 1) Total = 14

Currently this is the result, imagine 6-8 of these in your channel :scream: image

arkie commented 5 years ago

@SeaPlusPlush reopen if /roll mini 4d6k3 for Stat1, 4d6k3 for Stat2, 4d6k3 for Stat3, 4d6k3 for Stat4, 4d6k3 for Stat45, 4d6k3 for Stat6 doesn't work for you 👍

SeaPlusPlush commented 5 years ago

Wow, thanks for the super quick response. This is so much better, but there is a tiny problem in that it removes knowledge of what was being rolled from the dicebot (which cannot be edited or removed)

A user could do something like use a 6d6k3 image

And then edit it to look like it was 4d6k3 image

Yes there is an edit tag when doing so, but it might not be super apparent when checking a lot of peoples stuff (or if they are deleting their messages). I feel as though having an "impartial" way to confirm what was rolled is needed.

Also @arkie I cannot seem to reopen this issue.

SeaPlusPlush commented 5 years ago

Hello @arkie, I was wondering if you have had a chance to look at this again?

arkie commented 5 years ago

I feel like a proper fix would be to allow more customization in output instead of the present logic, where mini is checked as the prefix of the roll string and passed as a bool to the formatter which just removes the fields that would have been attached.

Definitely would probably be a good area to improve, but probably will not get a chance to do so in the near future.