changing system.json grid values format

[Vicnasc]

Description.

In FoundryVTT there's a warning that the fields gridDistance and gridUnit is deprecated in favor of a grid object with the distance and unit as inner fields. TemplateVersion is also getting a warning, but I'm not sure if its required for the system itself for documental purposes (couldn't find a version control over the code).

Motivation and Context.

It is not required, though may be useful for FoundryVTT's base guidelines.

Screenshots.

Types of Changes.

• Bug fix (non-breaking change which fixes an issue).
• New feature (non-breaking change which adds functionality).
• Breaking change (fix or feature that would cause existing functionality to change).

[snap01]

The current system supports FoundryVTT back to v10, these values are kept for backwards compatibility. A future update is planned for v12 that will drop both v10 and v11 support which will allow these and several warnings in game to be removed.

[ceriath]

Heya, i noticed the warnings today too and wanted to provide but a fix, but there is one already here.
To get rid of the warning while maintaining backwards compatibility you may simply put both in the system.json, see dnd5e for example: https://github.com/foundryvtt/dnd5e/blob/3.3.x/system.json#L213

[Vicnasc]

@snap01, thanks for the reply, my mistake for not looking at v10, only looked at v11

@ceriath, thanks for the suggestion, I applied the fix