AutarchLLC / foundryvtt-acks-core

Adventurer Conqueror King System Imperial Imprint (ACKS II) core game module for the Foundry Virtual Tabletop.
https://autarch.co
Other
6 stars 2 forks source link

Regression: Attacks need to use the proper type (melee or ranged) based on weapon #4

Closed ArcanistWill closed 2 weeks ago

ArcanistWill commented 5 months ago

Regression In the updated port, when you click on a weapon's icon to roll dice, it prompts to choose the attack type each time - as ranged or melee. This is a new bug that was not in v9 of the code.

To Reproduce Steps to reproduce the behavior:

  1. Add an Actor
  2. Add weapon that has only 1 attack type
  3. Click on the weapon icon to roll it
  4. You will see the error which is that the VTT does not recognize the weapon attributes.

Expected behavior When attacking with a Ranged (or Melee) weapon, it should not allow you to choose the other type. The VTT code should understand what attacks are possible based on the weapon.

Screenshots Here is an image of the unwanted behavior. attack-why-missile-etc

ArcanistWill commented 2 weeks ago

I am testing latest build with all recent fixes (12.0.15) and confirm this can be closed.