FoundryVTT-BetterRolls5e icon indicating copy to clipboard operation
FoundryVTT-BetterRolls5e copied to clipboard

Compatibility issue with Argon HUD

Open varXX404 opened this issue 2 years ago • 6 comments

Uncaught (in promise) Error: You must provide an _id for every object in the update data Array.
[Detected 4 packages: betterrolls5e, lib-wrapper, midi-qol, enhancedcombathud]
    _preUpdateDocumentArray https://foundry.mydomain.com/scripts/foundry.js:10185
    _updateEmbeddedDocuments https://foundry.mydomain.com/scripts/foundry.js:10152
    update https://foundry.mydomain.com/common/abstract/backend.mjs:153

This error seems to occur when Argon Combat HUD, Midi QOL, and BetterRolls are active together. Effectively it seems that Better Rolls blocks the messages from Argon when clicking one of the special action buttons (such as dodge, ready, dash, etc.), which prevents Midi from applying these conditions as DFreds Convenient Effects.

theripper stated that this was an issue on this side and outside his power, and asked for it to be added as an issue here https://github.com/theripper93/enhancedcombathud/issues/100

Please, a fix for this would be really nice.

varXX404 avatar Jun 20 '22 20:06 varXX404

Hi @varXX404, if you still need it, I've put up a release with this issue fixed over at my fork, including @cs96and's ammo fix.

Larkinabout avatar Jul 05 '22 13:07 Larkinabout

@Larkinabout awesome, thank you ! Any chance you can get in contact wiht @RedReign to officially take over the module if you are maintaining it ? Would love for more people to have access to the most up-to-date version of the module.

varXX404 avatar Jul 29 '22 09:07 varXX404

I'm not sure what's happening with this at the moment. Last I spoke to @CarlosFdez earlier this month, LukeAbby was working on a v10 version, but not maintaining the current version here yet. I'm sort of waiting for v10 as any effort I put into understanding and maintaining this version would probably overstep what's happening there and become redundant too.

Aside from that, I'm still actively using Better Rolls 5e myself, so would be keen to help out where I can.

Larkinabout avatar Jul 29 '22 12:07 Larkinabout

Hi @varXX404, if you still need it, I've put up a release with this issue fixed over at my fork, including @cs96and's ammo fix.

thnaks but how to install this fork?

Elthair avatar Jul 12 '23 09:07 Elthair

Note that BetterRolls is no longer being developed. Unless you are still on v9 of foundry, you should switch to Ready Set Roll

cs96and avatar Jul 12 '23 11:07 cs96and

Hi @varXX404, if you still need it, I've put up a release with this issue fixed over at my fork, including @cs96and's ammo fix.

thnaks but how to install this fork?

As cs96 said, you should be switching over to Ready Set Roll for anything past V9, as for installing it, I think you can just download it as a zip, go to the modules folder of your install and just overwrite the betterrolls folder with the one from the fork download

varXX404 avatar Jul 12 '23 15:07 varXX404