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

[BUG] When rolling an attack, the Ready-Set-Roll Chatmessage does not appear. #6

Closed
Morgeth opened this issue Sep 6, 2022 · 8 comments
Assignees
Labels
bug Something isn't working

Comments

@Morgeth
Copy link

Morgeth commented Sep 6, 2022

image
image

Description
Once using the character sheet to activate an attack, the damage roller instead rolls using the old visual style, however automatically rolls the dice regardless. A Chat message matching the message displayed in the readme, is not present at any point.

Steps to Reproduce

  1. Install using the Client search function.
  2. Activate on world
  3. Roll basic weapon or spell attack

Versions

  • Module: [1.0.1]
  • dnd5e: [2.0.2]
  • Foundry: [10.284]
@Morgeth Morgeth added the bug Something isn't working label Sep 6, 2022
@Morgeth
Copy link
Author

Morgeth commented Sep 6, 2022

Ive come to realise that the roll infact have the correct appearance however it does not automatically roll for the damage of the dice, bringing up the option to choose between critical and normal hit. If the button is pressed then it proceeds as normal. Not sure if that is intended or not. If it is, then I misunderstood. :)

@MangoFVTT
Copy link
Owner

MangoFVTT commented Sep 6, 2022

@Morgeth it seems like you may have accidentally triggered the default roll dialog in some fashion. Are you clicking the item's image to trigger the quick roll? And can you confirm if the Enable Quick Rolls for Items setting is active and you are not holding alt?

image

@Morgeth
Copy link
Author

Morgeth commented Sep 6, 2022

image
Enable quick Roll for items is active. Not pressing alt,

I am using the Tidy5e sheet and pressing the image in order to activate the item.
image
When I do so, It automatically rolls the d20 in order to attack, and immediately gives me the standard damage roll prompt (Example below). During this time, the prompt has the default appearance.
image

If I press normal or Critical, then the chat message is updated with the correct appearance
image

@Morgeth
Copy link
Author

Morgeth commented Sep 6, 2022

Going by the basis of Better rolls, I presume that the damage is meant to roll automatically rather than giving me the prompt for normal or critical damage?

@MangoFVTT
Copy link
Owner

That is correct, however I haven't tested this module in conjunction with any other modules just yet, and I don't use TidySheet5e myself personally. I will investigate this to see what the issue might be

@MangoFVTT
Copy link
Owner

@Morgeth I tested this out with TidySheet and was unable to reproduce the errors you are getting. Are you still having the problem? If yes, what other modules do you have installed and do any of them interact with dice rolls?

Further to that, if you are not having this problem with every item, exporting the items that you are having the issue on and posting them here so I can take a look at their data would be very helpful.

TidySheet

@Morgeth
Copy link
Author

Morgeth commented Sep 8, 2022

Hello again Mango, sorry for not replying back yesterday but I wanted to get some further research done into it myself so I could come back with more concrete information on what is causing it.

I managed to find the issue, namely the thing causing the error was when the plugin was used in conjunction with Midi-QoL. Midi QoL has an QoL change for rolling automation that ends up conflicting with the plugin. When the option is disabled, the dice works as expected.
image

image

@MangoFVTT
Copy link
Owner

MangoFVTT commented Sep 8, 2022

Ah that explains it. Midi QOL modifies the same functions as this module, so you cannot use both in conjunction (for dice automation). You have to pick one or the other. Hope that helps you solve the issue 😄

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

No branches or pull requests

2 participants