DiscordSRV

DiscordSRV

86.8k Downloads

Define server admin in config.yml and refer to it as a placeholder

metalshark opened this issue ยท 3 comments

commented

Being able to define the primary server admin in config.yml then refer to it in:

messages.yml
ServerWatchdogMessage

with it having a default of %guildowner% may cut down on future support requests. Admins can still bespoke the ServerWatchdogMessage or add more than one server admin (using the <@123> format).

commented

Sounds a bit uneccasery, it's just 2 places.

with it having a default of %guildowner% may cut down on future support requests.

What?

commented

May be unnecessary, however, I asked the question when starting out and so did another user (e.g. how do you specify who the watchdog message goes to). I am using the Placeholder API to send a few more messages and would be nice to be able to define the Discord admin in one place to use elsewhere. Means more of the config is portable between systems and all the Discord stuff is defined in the Discord configs.

I do things like notify staff from the lagassist plugin when TPS drop, etc and would be nice not to hardcode their account IDs on each system.

commented

I'm not particularly a fan of this or the reason provided for the change. Having an entire configuration option that is only utilized in one message as opposed to just being in the message is unnecessary. While I can see the point of view that the config and messages files can be separated, realistically, it's not necessary for such a small use.
#645 (comment)