pf2e
pf2e copied to clipboard
Add siege weapon actor type
Adds a siege weapon actor and a sheet modeled off of the vehicle sheet style. Icon is derived from https://game-icons.net/1x1/heavenly-dog/catapult.html
This isn't something we can consider for this release, and next major release will be for V10--which will require a full rewrite of the PR.
This isn't something we can consider for this release, and next major release will be for V10--which will require a full rewrite of the PR.
I figured that might be the case for this type of enhancement given the focus on v10. I'll plan to update the PR to the v10 release when available.
OK, sounds good. Thank you for the contribution, though, and sorry about the timing!
I'm resolving a UI issue and will fix the eslint fallout along with that.
This should pass CI now if you can approve the workflow.
The only concern I have about merging this is that I'm now not happy with the data model of storing mounted siege weapon stats as actions. I started working on converting this to have a fixed aim, load, launch set of data which I think is better for future system automation. e.g. launch would have traits, damage, target type, and save DC properties. In the current model it's just an unstructured wall of text in the Launch action. It's not a huge amount of data entry, but it would all need to change for this data model update.