search

kandashi / Next-Up

Auto focus on new combatant
MIT License
12 stars 9 forks source link

[BUG] Current NPC HP reset after next turn #102

Closed mech-tools closed 3 years ago

mech-tools commented 3 years ago

Hello,

FVTT 0.8.6 Nextup: latest Browser: Chrome Dnd: 1.3.5

When I remove HP from a NPC token, and it is that NPC turn, when pressing the NEXT TURN button, the NPC goes back to its previous HP count.

https://user-images.githubusercontent.com/1687854/123014386-78e21580-d3c6-11eb-9117-3fc4ae7a13d3.mp4

kandashi commented 3 years ago

Is this will no other modules enabled ?

mech-tools commented 3 years ago

@kandashi nope, only next up module.

kandashi commented 3 years ago

Can you check without any modules, I'm not sure how next up would be triggering this

mech-tools commented 3 years ago

@kandashi made the test, no issue without any modules. Note that the demo video showcases a combat without any PCs, only NPCs, could it be smt in relation with combats without PCs?

mech-tools commented 3 years ago

@kandashi made the test with a PC as well in the combat. Same issue.

mech-tools commented 3 years ago

@kandashi tested in a new world with only next up & libwrapper (as suggested by the module). No prob at first. Then, after matching my world configuration, trouble started. Maybe if you match this exact same config, you will have the issue? image

Note that panning to next combatant doesn't work with this configuration.

kandashi commented 3 years ago

Well, I can replicate, now to figure out whats up

kandashi commented 3 years ago

Found the problem, fix going live soon