Closed GravityWolfNotAmused closed 2 years ago
@Specker I still need to add docker changes, I want to clean it up, but I at least wanted to see if someone would test it out while I accomplish that.
Updated Snapshot:
@Specker I have decided that it's not worth the extra time to revisit for a few low-hanging fruits. So, I have brought support for Steam, CFX, Scum, and Minecraft.
I might also bring in Battle Metrics support to have a broader range of variations without the need for unique logic.
@Specker
I have moved the logic for getting docker configuration. Look here: https://github.com/GravityWolfNotAmused/PlayerCountDiscordBot/blob/CFX-Update/DiscordPlayerCountBot/Configuration/DockerConfiguration.cs
@Specker I think have covered everything for ProviderType in the logic, and docker files. Can you test the docker stuff for me please? What else needs to be done to release this thing?
For anyone who wishes to test out the CFX, Scum, or Minecraft features you can use the snapshot attached to this comment.
Please note that I am not doing self-contained executables, so there are now fewer files with releases.
@Specker Any chance you could look into automating the updating process of the docker image?
We are now working off of the dev branch, and promoting to master.
Providers are classes that return required data to update the bots status (IServerInformationProvider)
Available Providers:
Depending on which provider is chosen then the difference in logic will execute but provide the same resulting data. The goal for after this PR is complete is to create more providers for different games that are not supported.