search

League-of-Foundry-Developers / foundry-vtt-types

Unofficial type declarations for the Foundry Virtual Tabletop API
MIT License
112 stars 52 forks source link

Update `ActiveEffect` to v12 #2697

Closed NekroDarkmoon closed 1 month ago

NekroDarkmoon commented 1 month ago

Updates ActiveEffect to v12.

Notes:

Changes 

get target(): Actor | null;

to 

get target(): Document | null;

There was a TODO there to make the return type more generic. There are a bunch of systems that specifically target Item and TokenDocument.

LukeAbby commented 1 month ago

Why not Item | Actor | TokenDocument? Are there systems that target, say, ActiveEffect or FogExploration?