bayashi / mclocks

Free desktop clock for multiple time zones๐Ÿ•’๐ŸŒ๐Ÿ••
Other
10 stars 4 forks source link

Feature request: make the program inform the user when the config file has an error #91

Open LinuxOnTheDesktop opened 2 years ago

LinuxOnTheDesktop commented 2 years ago

Thank you for your software.

My config file as as follows.

{
    "clocks": [
        { "name": "Auckland", "timezone": "Pacific/Auckland"}
        { "name": "Melbourne", "timezone": "Australia/Melbourne" }
    ],
    [. . . snip . . .]
}

In that configuration there is a comma missing after { "name": "Auckland", "timezone": "Pacific/Auckland"}. Instead of telling me of the error, the program just reverted to its initial configuration. That left me baffled - until I discovered the missing comma. So, some sort of error message would be useful!

I am using mclocks 0.1.15 (downloaded from MajorGeeks.com), on Windows 10.

bayashi commented 1 year ago

Thank you for your nice feedback. I 100% agree. Could you please use JSON linter in your editor for config file until I improve an error like this?

LinuxOnTheDesktop commented 1 year ago

Certainly.