add moderation class, new cmd /moderation, and fix cmd output #50
Conversation
ghost
mentioned this pull request
| @@ -248,6 +252,7 @@ class CabalDetails extends EventEmitter { | |||
| */ | |||
| addStatusMessage(message, channel=this.chname) { | |||
| if (!this.channels[channel]) return | |||
| debug(channel) | |||
| @@ -43,6 +43,14 @@ module.exports = { | |||
| }) | |||
| } | |||
| }, | |||
| share: { | |||
| help: () => 'print a cabal key with you as admin. useful for sending to friends', | |||
There was a problem hiding this comment.
Discussion point: do we want /share to share with you as admin, or with you + whoever is in your admin/mod-keys?
There was a problem hiding this comment.
@noffle good discussion point! my reasoning was as follows:
what i reckoned was that it would get so absurdly long as to be useless if we shared all the admin + modkeys, which is why i opted for sharing oneself as an admin; reasoning that would have a similar effect
(at least that was what i was thinking at the time; given that the passed-in admin/modkeys don't actually issue admin/mods as statements then they won't be transitively be available)
There was a problem hiding this comment.
Hey! I wasn't sure where to continue this convo :)
As I understand it, most people have one admin key. So a /share that used your admin keys should also just include one key, right? It feels like it might get confusing if inviting another user gives them a different moderation view than yours.
this pr moves some of the moderation commands from
commands.jsto a new class, and exposes a bunch of methods.the intent is for clients other than the terminal client to be able to easily use moderation commands. e.g. cabal desktop can tie menu actions to moderation commands more easily
details.moderation, see moderation.js for the full listing atm/moderationis a new command for showing moderation related commands/helphas been fixed by making use of monotonic timestampscommand:details.on('command', ({ command, arg })