Page MenuHomePhabricator

Make it possible to trigger commands to -help from -helpers
Closed, MigratedPublic

Assigned To
Authored By
DatGuy
Dec 29 2016, 1:52 PM
Referenced Files
F6367: Redirection.png
Jan 2 2017, 3:25 PM

Description

Samwalton9 and yeryry suggested this. Not sure how it'll be integrated. My suggestion would be to add a third param for a channel (so !command > user #channel or !command #channel).


  • Storage and configuration ability for cross-channel command authorisation
  • Per-command setting for default target (such that !helper always triggers in -helpers)
  • Target adjustment for command redirection, I agree that > #channel is probably the best, as an extension of the command redirection syntax: Redirection ::= '>' ' '? Nickname? Channel?
  • CommandHandler or similar handling to redirect messages based on authorisation
  • Access control changes - this is very much aimed at single-channel infrastructure currently.

Event Timeline

So this will probably be quite hard.

-help and -helpers aren't special channels for the bot in any way, other than for one command (!helper) which has these channels hard-coded - there's an existing task for making this configurable which will probably need to be fixed first, as this is related.

Things that I think need to happen:

  • Storage and configuration ability for cross-channel command authorisation
  • Per-command setting for default target (such that !helper always triggers in -helpers)
  • Target adjustment for command redirection, I agree that > #channel is probably the best, as an extension of the command redirection syntax: Redirection ::= '>' ' '? Nickname? Channel?
  • CommandHandler or similar handling to redirect messages based on authorisation

As a quick estimate, 2) will take at most 30 mins, 4) will take about an hour, 1) will take a few hours due to necessary schema design and changes, and 3) will probably take a couple of hours too, due to the testing requirements.

Command redirection syntax:

Redirection.png (68×444 px, 3 KB)

stwalkerster claimed this task.