Skip to content

Set Exploration Activity macro not working in V12 #17

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
thejoester opened this issue Aug 15, 2024 · 2 comments
Open

Set Exploration Activity macro not working in V12 #17

thejoester opened this issue Aug 15, 2024 · 2 comments

Comments

@thejoester
Copy link

thejoester commented Aug 15, 2024

In Foundry V12 the macro for Set Exploration Activity is not working. Throwing an error in the console:

Uncaught (in promise) Error: You are accessing the CharacterPF2e "data" field of which was deprecated in v10 and replaced with "system". Continued usage of pre-v10 ".data" paths is no longer supported"
[No packages detected]
    get data http://<my-ip>:30000/scripts/foundry-esm.js:12036
    tokens http://<my-ip>:30000/scripts/foundry.js line 20071 > AsyncFunction:4
    anonymous http://<my-ip>:30000/scripts/foundry.js line 20071 > AsyncFunction:3
    #executeScript http://<my-ip>:30000/scripts/foundry.js:20076
    execute http://<my-ip>:30000/scripts/foundry.js:20022
    #onExecute http://<my-ip>:30000/scripts/foundry.js:79355
foundry-esm.js:12036:15
@thejoester
Copy link
Author

So I found the fix to this issue:

in the macro you change Line 2 from

['character'].includes(t.actor.data.type),

to

['character'].includes(t.actor.type),

I created a pull request to merge this change: #18

@suntinwong
Copy link

Encountered same issue, modifying the macro on Line 2, as specified above works. Please approve his PR and get it into main!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants