tposney / midi-qol

Other
5 stars 0 forks source link

Error in DamageOnlyWorkflow #713

Closed tposney closed 1 year ago

tposney commented 2 years ago

In GitLab by @renato.innocenti on Feb 24, 2022, 16:38

there is an error in the workflow it is possible to detect it using the divine smith of the compendium. the problem with the targetToken.document.uuid line as it already passes the document as targetToken, there is no "document" property. I solved it by removing this property from the requested lines, but it would be nice to fix it in the next update

tposney commented 2 years ago

I did notice a problem with that the other day. However, I can't tell from you report which lines you are referring to. I assume you mean the divine smite item, there is no targetToken.document.document.uuid. If you mean in damageOnlyWorkflow, I think they were cleaned up in the latest release.

If not I've just done a pass through the whole code base to try and make sure there are none left.

In any event it the error is still there please indicate which line of code you are referring to.