Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Compact Version or way to hide #47

Open
SeaPlusPlush opened this issue Mar 5, 2019 · 4 comments
Open

Compact Version or way to hide #47

SeaPlusPlush opened this issue Mar 5, 2019 · 4 comments

Comments

@SeaPlusPlush
Copy link

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 😱
image

@arkie
Copy link
Collaborator

arkie commented Mar 5, 2019

@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 👍

@arkie arkie closed this as completed Mar 5, 2019
@SeaPlusPlush
Copy link
Author

SeaPlusPlush commented Mar 5, 2019

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.

@arkie arkie reopened this Mar 5, 2019
@SeaPlusPlush
Copy link
Author

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

@arkie
Copy link
Collaborator

arkie commented Apr 6, 2019

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants