fvtt-module-lmrtfy icon indicating copy to clipboard operation
fvtt-module-lmrtfy copied to clipboard

Feature request: Don't await actor[rollMethod].call in default case (line 328 lmrtfy.js)

Open tposney opened this issue 1 year ago • 0 comments

Only in the default case is the roll awaited, this has the side effect of leaving the dialog open until the dice so nice 3d dice rolls complete. Also can you pass the lmrtfy flags data as options to the roll call - that would solve a timing issue I created where many messages get the same tag.

This has caused some confusion amongst users of my modules. line 328 or lmrtfy.js

await  actor[rollMethod].call(actor, ...args, options) 

becomes

setProperty(options, "flags.lmrtfy", {"message": this.data.message, "data": this.data.attach})
actor[rollMethod].call(actor, ...args, options)

tposney avatar Jun 03 '23 05:06 tposney