make issuer of CHANNEL command first admin of new channel
context
- in !528 (merged) we automated channel creation
- this is great, but sometimes users mistype their own phone number
- so... let's automatically make the issuer of a
CHANNEL
command an admin (preventing typos from stopping channel launch) - optionally... let's consider whether to use this issue to enforce a requirement for channels to have at least 2 admins by keeping the requirement that
CHANNEL
commands have a payload and requiring that the number in the payload be different than the command issuer
behavior
(stub!)
happy path
- when i issue a
CHANNEL
command that executes successfully, i am automatically made the first admin of the channel - if i list my own number, i am still only made admin once
- if i provide phone numbers other than mine they also made admins
sad paths
- optionally... if i list no numbers other than my own i get an error message
- if i list invalid numbers, i get an error message
discovery
- explain how this works in both
HELP
andDOCS
implementation notes
- read the sender of the
CHANNEL
command and make them the first admin increateChannel
- dedupe the payload of
newAdmins
and remove command sender if it is part of it - optionally... after deduping, ensure that
newAdmins
has at least 2 elements
open questions
- should we collapse 2-admin requirement into this card or make it its own card?