Skip to content

Add a new mod [Calendar and Weather]#2073

Merged
kfroll20 merged 12 commits into
Roll20:masterfrom
jbabin49:master
Jun 13, 2025
Merged

Add a new mod [Calendar and Weather]#2073
kfroll20 merged 12 commits into
Roll20:masterfrom
jbabin49:master

Conversation

@jbabin49

Copy link
Copy Markdown
Contributor

Add a new mod in Roll20 API scripts.
This mod is an entire customizable calendar and weather in english and french.

@kfroll20

Copy link
Copy Markdown
Contributor

Hello @mailare49!

Thank you for contributing to the API scripts repo. Everything looks good to publish, but we will need a subfolder for your version as outlined in the documentation.

As this is the first version, you can go ahead and make a subfolder named 1.0 with the Calender and Weather.js file in it :)

Thanks!

@jbabin49

Copy link
Copy Markdown
Contributor Author

Hello @kfroll20, thanks for fast answer. I have create the subfolder to put the mod inside.

@kfroll20 kfroll20 self-requested a review June 12, 2025 18:26

@kfroll20 kfroll20 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

The JSON validator is failing because dependencies in your script.json file is an empty string when it needs to be a list

"dependencies": []

@jbabin49

Copy link
Copy Markdown
Contributor Author

Hello, I have correct this.

@kfroll20

Copy link
Copy Markdown
Contributor

Perfect, thank you! Just so you're aware, there are a great number of websites down due to a google outage so I am holding off on doing any merges until it has been resolved. Otherwise this looks good to go so I will merge once we have been given the greenlight!

@jbabin49

Copy link
Copy Markdown
Contributor Author

Thanks for informations

@kfroll20 kfroll20 merged commit 820e59d into Roll20:master Jun 13, 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.

2 participants