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

DF Manual Rolls: Basic Dice roller option unsupported #464

Open
MotoMoto1234 opened this issue Mar 31, 2023 · 0 comments
Open

DF Manual Rolls: Basic Dice roller option unsupported #464

MotoMoto1234 opened this issue Mar 31, 2023 · 0 comments
Assignees
Labels
bug Something isn't working

Comments

@MotoMoto1234
Copy link

Module
DF manual rolls 2.4.0, libwrapper 1.12.12.0

Describe the issue
The dice roller supports the code [[1d4x]]. This causes any 4's to have an additional die rolled. In the core roller DF manual rolls does not do anything with the additional die, its rolled automatically. In Midiqol, DF manual rolls is prompted for another die roll. So something in core DF can't work with but midi somehow fixes it for you.

To Reproduce
Steps to reproduce the behavior:

  1. Change the dnd5e spell healing words formula from 1d4 to 1d4x
  2. It won't prompt you a second time for the next roll

Expected behavior
In midi after you enter the 4 in the dice roll, it prompts you for the exploded dice's roll then. In core it just shows the card with the rolled dice rolling the second for you, if it explodes, it also is auto rolled not manually.

Screenshots (Optional)
If applicable, add screenshots to help explain your problem. If you hit the [F12] key while in Foundry, you will see a window appear. Open the "Console" tab (if it's not already open). If you scroll through this, it is a log for the page. Anything that appears in RED are errors. You can screenshot those errors and add them to this ticket.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants