-
Notifications
You must be signed in to change notification settings - Fork 16
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
Use correct token id reference for combatant #77
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We just tested this change and this indeed fixes the bug #70
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Fixes #70 with VTT 0.8.8 and DnD5e 1.3.6
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
checked and this change fixes issue #70
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this fixes issue #70
Fixes #70 with VTT 0.8.8 and PF2E 2.9.1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This change seem to fix the prolem in all tested circumstances (fresh world with ds4 and turnmarker only, 2 running campaign worlds with multiple active other modules). Marker is seen and correctly positioned after applying this patch manuel to the installed module. (VTT 0.8.8 with ds4 1.2.1)
I've merged this change into main already from another PR. Sorry, been pretty busy at work, and never got the time to come back and nail down this bug. It should be working on the current release. It was part of this larger PR. |
This resolved #70 for me.