FM-17 / poglink

A bot that integrates the ARK Web API with Discord.
MIT License
12 stars 4 forks source link

Add ability to provide custom url/embed title pairs #72

Open FM-17 opened 1 year ago

FM-17 commented 1 year ago

A hotfix was done to account for the division of ATLAS endpoints:

PVE: http://atlasdedicated.com/atlas-eu-pve/dynamicconfig.ini PVP: http://atlasdedicated.com/dynamicconfig.ini

I added an if statement that checks for atlas and pve in the endpoint, then skips parsing the DEFAULT_SERVER_INFO dict (which is used to match endpoints to embed titles), and overrides the title/colour to hardcoded values.

This fix needs to be replaced by allowing the user to input custom endpoint/embed title pairs into the config. In fact, we could replace the DEFAULT_SERVER_INFO parsing section altogether, and just make it so these titles are required config variables.