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] Fail falgs not working as expected #1

Closed
mech-tools opened this issue Jan 11, 2022 · 2 comments
Closed

[BUG] Fail falgs not working as expected #1

mech-tools opened this issue Jan 11, 2022 · 2 comments
Labels
documentation Improvements or additions to documentation

Comments

@mech-tools
Copy link
Contributor

mech-tools commented Jan 11, 2022

Hello,

FVTT 9.242
adv-reminder 0.2

Thx for this module. This is awesome.
I, like you, play with a little automation but not as much as Midi Qol.
This module really comes in handy. I was surprised to find this module working like a charm with BetterRolls5e.

When I set a Fail Midi Flag, only the saving throws are affected.
Actually, always the saving throws, never the skill throws or ability throws. (tested with & without BetterRolls5e)

EDIT: tested almost all Midi Qol Flags with BetterRolls5e. The only one not working is CriticalReminder.

@kaelad02
Copy link
Owner

Thanks for giving this a shot and glad to hear it works (mostly) with BetterRolls5e considering I hadn't tested with it.
You're right, only the fail flags for saving throws are implemented currently. I know Midi has fail flags for lots of checks but I didn't implement them because I didn't see any conditions or effects that would actually auto-fail them. I know Stunned will auto-fail Strength and Dexterity saving throws but I couldn't remember anything else auto-failing ability checks or any of the others. But if you want them, it's not too much work to add them.

@mech-tools
Copy link
Contributor Author

@kaelad02 no particular use-case here. Just thought it was a bug :)

@kaelad02 kaelad02 added the documentation Improvements or additions to documentation label Jan 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

No branches or pull requests

2 participants