search

vkdolea / age-system

Unofficial implementation of AGE System (Green Ronin) for Foundry VTT
Apache License 2.0
12 stars 16 forks source link

Feature Request (low priority): Integrate AGE (Unofficial) with Token Action Hud Core #92

Open r2roberts opened 1 year ago

r2roberts commented 1 year ago

You create a new module that connects Token Action Hud Core with your intended game system. No code should be needed in the game system, nor TAHC (in principal).

https://foundryvtt.com/packages/token-action-hud-core https://github.com/Larkinabout/fvtt-token-action-hud-core/wiki/Developing-a-System-Module

vkdolea commented 1 year ago

I have interest but no plans to integrate the system with Token Action HUD (or Core). Maybe in the future it could be executed. I will leave this issue open to track the request.

r2roberts commented 1 year ago

Would you mind if I pulled your repository to take a look at what would be needed to get a rudimentary start on integrating Token Action Hud with your system? I'm not guaranteeing anything, but I would like to see what I could do to help.

vkdolea commented 1 year ago

Fell free to clone the repo and attempt to implement. If I recall correctly, Token Action HUD module would host all necessary code without any need to tweak the system code. However, if making small changes on the system make the life easier to add TAH compatibility, fell free to submit a PR.

r2roberts commented 1 year ago

They've changed the way they support systems. It's my understanding that you don't touch TAH's code any more. You create a separate module that references both the API of TAH and the API of the target system. I think if more of your system was open in a public API, like your "game.ageSystem.rollOwnedItem("Item Name", false)" macro, then integration would be doable. I'm pretty busy now, but I will probably put in a pull request soon.