Joystream / discord-bot

Joystream Discord Bots
GNU General Public License v3.0
0 stars 4 forks source link

Tip bot requirements #6

Open bedeho opened 1 year ago

bedeho commented 1 year ago

Background

People have expressed an interest in being able to socially tip each other on Discord, right now do not have a way to do that. There has been an attempt at doing that, but it has some significant design flaws - among which is to require users to provide their seeds, so we need some more solid requirements.

Note: I am not conviced about the real value in executing on this now, given how complex it is, and likelihood of it being adopted, but just posting this

Proposal

Maintiner

@DzhideX

Infra

Jsgensis operated.

Target chain

Carhtage

Requirements

The idea in this proposal is to only use Discord handles as identifiers, and sidestep the entire problems of mapping on-chain memberships to users, as that gets very messy and complex. This requires that Discord handles are used as identifiers, and that we are sure that they are obth immutable and unique. This seems to be changing on Discord these days, so we must be careful to get it right.

Resource

https://github.com/paritytech/substrate-tip-bot

ivanturlakov commented 11 months ago

Tested on https://discord.gg/T2jspUcb create_discord_server

✅ /help Please follow this link to learn how to use the tipping bot. https://www.notion.so/joystream/JOY-tipping-bot-cc69c748e3994234842d03434facc82c?pvs=4

⚠️ We need to add instructions on how to top up the balance. It would be great to have all the information you need inside the dialog with the bot, eliminating the need to constantly go to the link to Notion.

Ivan Deposited 3JOY to pool 5EX1JGrkV84M7RLbaaYX4DSrNLC9ZSpfNU7TtobEaF2UMtxC

✅ /status Bot version : 1.0.0 Pool Wallet : j4TmusK6iMqZNx2nLrYzCuN9qtiGj4LPq6QjuGMdZ1KB71b9D Pool Amount : 40.99JOY ⚠️ We need to unify pool addresses formats to avoid confusion (help info and /status)

✅ /verifydeposit1 Go to this URL https://polkadot.js.org/apps/?rpc=wss://rpc.joystream.org:9944#/signing and sign the following data with the given account. eSLbatPdk8

✅ /verifydeposit2 Your wallet has been verified. Your deposit is 3 JOY

✅ /getbalance Ivan @Ivan balance is 3JOY The wallet address bound to this discord account is ...

✅ /getbalance Create @Create balance is 7JOY The wallet address bound to this discord account is ...

✅ /send to Create 1 JOY You have sent @Create 1 JOY

✅ /getbalance Ivan @Ivan balance is 2JOY The wallet address bound to this discord account is ... ⚠️ I suggest to add @ symbol to Input label in cases like this

Screenshot 2023-11-08 at 12 16 18

✅ /getbalance Create @Create balance is 8JOY The wallet address bound to this discord account is ...

✅ /withdraw 1JOY Ivan You have withdrawn 1 JOY

✅ /getbalance Ivan @Ivan balance is 0.99JOY The wallet address bound to this discord account is ...

ivanturlakov commented 11 months ago

⬆️ Updated my report and added some thoughts. Overall all is good with the basic functionality 👍