Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

improve UX of bot's "sendMoney" command #48

Open
2 tasks
Sladuca opened this issue Apr 30, 2021 · 0 comments
Open
2 tasks

improve UX of bot's "sendMoney" command #48

Sladuca opened this issue Apr 30, 2021 · 0 comments
Labels
ramp up task Very simple, often trivial task perfect for newcomers ux

Comments

@Sladuca
Copy link
Contributor

Sladuca commented Apr 30, 2021

Currently sendMoney is very crude and doesn't yet have support for NE5 Tokens, even though we eventually want it to. We need to re-structure the command so that it works.

In the bot subrepo, in commands/Send.ts:

  • rename "sendMoney" command to "send"
  • add an error message to "send"command that says something along the lines of "unsupported asset type" whenever the asset argument given is not supported. For now this error should happen for every asset that isn't $NEAR.

In the bot subrepo, in config/config.ts, change the current activity of the bot. name can be literally any string, and type can be one of the strings listed here

@Sladuca Sladuca added ramp up task Very simple, often trivial task perfect for newcomers ux labels Apr 30, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ramp up task Very simple, often trivial task perfect for newcomers ux
Projects
None yet
Development

No branches or pull requests

1 participant