Skip to content

Conversation

@Dedalus36
Copy link
Contributor

Adding a new optional API to accompany the Grimwild character sheet, enabling enhanced roll outputs.

Adding a new, optional, API to accompany the Grimwild character sheet, enabling enhanced roll outputs.
@Alicekb
Copy link
Contributor

Alicekb commented Oct 7, 2025

Hey @Dedalus36, it looks like there is an issue in your script.json that caused the validation step to fail. Please update this PR with a fix and I can get this merged as soon as possible.

@Dedalus36
Copy link
Contributor Author

Thanks for the heads up, @Alicekb! Was just missing a comma, should be good to verify now.

"useroptions": [],
"dependencies": [],
"modifies": [],
"conflicts": [],
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Trailing commas aren't allowed in JSON

Suggested change
"conflicts": [],
"conflicts": []

@NorWhal
Copy link
Contributor

NorWhal commented Oct 9, 2025

I think the above is the issue that needed fixing, @Dedalus36

@Dedalus36
Copy link
Contributor Author

Hi @NorWhal and @Alicekb,

I've reviewed the script.json again and think the original issue might have been including the modifies field as an array rather than an object. That change, and the incorrect edit I made to the comma that NorWhal flagged, have now been corrected. Let me know if additional edits are needed. Thanks for your time!

@Dedalus36 Dedalus36 requested a review from NorWhal October 22, 2025 18:54
@Alicekb Alicekb merged commit 6d7d09c into Roll20:master Oct 28, 2025
1 check passed
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

Successfully merging this pull request may close these issues.

3 participants