-
Notifications
You must be signed in to change notification settings - Fork 7
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
V9 update needed for this line #1
Comments
Thank you for advising me on the syntax change in V9. I had not tried the script in v9 as I was still on v8 when I used it. I will evaluate the change you have suggested and let you know. |
The module itself should work fine in v9 however. The script is only if you want to avoid using the content presented in the module. |
Works great! Thank you for sharing! |
@datanerd3000 sorry about the delay in the fix. I have been very busy and have not had time to check that the fix suggested works. I have now verified that and updated the script appropriately, thank you for your suggestion. |
Waterdeep-City-of-Splendors/scripts/waterdeep.js
Line 26 in acaf97f
With foundry v9, they've updated the underlying process. I've found this works with v9
canvas.scene.createEmbeddedDocuments("Note", noteDataResolved);
instead ofcanvas.notes.createMany(noteDataResolved);
The text was updated successfully, but these errors were encountered: