GravityWolfNotAmused / PlayerCountDiscordBot

A simple discord bot that updates their status' to show player counts.
MIT License
36 stars 16 forks source link

1.0.9.3 - Docker Fix & Test Cases #46

Closed GravityWolfNotAmused closed 2 years ago

GravityWolfNotAmused commented 2 years ago

After the last update, I left an issue with the parsing of docker configuration variables for Application variables. This fixes the possible null, the option of having steam or battle metrics, or none.

I have built test cases around this parsing logic, so that in the future if I break something I know before releasing it. Please note that this issue doesn't extend to the standard configuration as JSON holds a specific format for dictionaries that would cause the parser to fail.

I also moved the logic for setting the channel name to an extension method.

Also! I have moved all of the documentation to the wiki. It's easier that way ;)