correct command for linking wolves acc #31

Merged

esy merged 2 commits from update-dm into main

2025-06-16 13:07:55 +00:00

esy commented

2025-04-23 11:30:40 +00:00

Owner

No description provided.

esy added 1 commit 2025-04-23 11:30:40 +00:00

correct command for linking wolves acc 22ff91b152

esy commented

2025-04-23 11:32:44 +00:00

Author

Owner

Might be better to store the command used somewhere and use something in CI (sed?) to keep it consistent with the docs/User.md, theres also the case where servers have to keep their channel descriptions updated e.g on games dev its still the old /link, and there is chat history saying its other stuff.

requested review from silver

2025-04-23 11:33:07 +00:00

esy commented

2025-04-23 11:35:52 +00:00

Author

Owner

@esy wrote in #31 (comment):

theres also the case where servers have to keep their channel descriptions updated e.g on games dev its still the old /link, and there is chat history saying its other stuff.

For this if the bot is given sufficient perms it can probably be automated pretty easily if layout of the channels for member verification / bot commands is standardized - e.g compsoc used bot_commands, game dev used member_verification

@esy wrote in https://forgejo.skynet.ie/Skynet/discord-bot/pulls/31#issuecomment-2260: > theres also the case where servers have to keep their channel descriptions updated e.g on games dev its still the old /link, and there is chat history saying its other stuff. For this if the bot is given sufficient perms it can probably be automated pretty easily if layout of the channels for member verification / bot commands is standardized - e.g compsoc used bot_commands, game dev used member_verification

esy added 1 commit 2025-04-23 11:39:53 +00:00

fix: out of date command 96cc5e4a84

esy force-pushed update-dm from 96cc5e4a84 to d28a56f255

2025-04-23 11:40:45 +00:00

Compare

silver commented

2025-06-16 05:24:37 +00:00

Owner

Only seeing this now.

theres also the case where servers have to keep their channel descriptions updated e.g on games dev its still the old /link

Personally I would rather the bot didnt have the ability to do that, keep it as stupidly simple as possible.
I have been saying taht folks should link to https://forgejo.skynet.ie/Skynet/discord-bot/src/branch/main/doc/User.md since that is going to be kept up to date (and easier to centrally manage).
That being said it might be worth putting it in the Committee.md

and there is chat history saying its other stuff.

Ye, that is completely on me, joys of rough early days and them basically being teh beta testers.

Feel free to merge this in whenever, and thank ye

Only seeing this now. > theres also the case where servers have to keep their channel descriptions updated e.g on games dev its still the old /link Personally I would rather the bot didnt have the ability to do that, keep it as stupidly simple as possible. I have been saying taht folks should link to https://forgejo.skynet.ie/Skynet/discord-bot/src/branch/main/doc/User.md since that is going to be kept up to date (and easier to centrally manage). That being said it might be worth putting it in the Committee.md > and there is chat history saying its other stuff. Ye, that is completely on me, joys of rough early days and them basically being teh beta testers. Feel free to merge this in whenever, and thank ye

silver approved these changes 2025-06-16 05:44:15 +00:00

silver left a comment

Owner

LGTM

esy merged commit 8560ed6de5 into main

2025-06-16 13:07:55 +00:00

esy referenced this pull request from a commit

2025-06-16 13:08:02 +00:00

Merge pull request 'correct command for linking wolves acc' (#31) from update-dm into main