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

When a number command is expected but other is issued (/orange for example) #47

Open
lenara opened this issue Apr 20, 2017 · 2 comments
Assignees
Labels

Comments

@lenara
Copy link
Collaborator

lenara commented Apr 20, 2017

This bug happens when the user chooses a flavor to invest on, the list with suggested investment amounts is displayed, but instead of choosing an amount the used sends again a flavor command or any other command for that matter.

The crash is quite bad and will also crash again on restart unless the user types a value command.

@lenara lenara added the bug label Apr 20, 2017
@lenara lenara changed the title Bug when choosing flavors to invest on When entering the "numbers" section, first make sure what was typed was a number Apr 25, 2017
@lenara
Copy link
Collaborator Author

lenara commented Apr 25, 2017

this bug affects other sections when the user has to select a command with a number (list from gift, creative, etc...)
if the user types another command instead, things will happen.
in the market, the bad loop crash mentioned in previous comment
in the creative/gift , if the user types /orange it will "switch" to the market bets. it will only crash once, not loop-crash

@lenara lenara changed the title When entering the "numbers" section, first make sure what was typed was a number When a number command is expected but other is issued (/orange for example) Apr 25, 2017
@lenara
Copy link
Collaborator Author

lenara commented Apr 25, 2017

Fixed for speculation market. Used handy isNaN() function.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests

1 participant