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

Hexadecimal number can be used to specify amount for send/receive commands [develop] #1442

Closed
annadanchenko opened this issue Jul 12, 2017 · 2 comments · Fixed by #8128
Closed

Comments

@annadanchenko
Copy link

annadanchenko commented Jul 12, 2017

Description

Type: Bug

Summary: If hexadecimal number (e.g. 0x12) is provided as amount to send/receive commands then "Amount is not valid number" error is expected. Currently, number is accepted as valid one.

Expected behavior

"Amount is not valid number" error should be shown if provide hexadecimal number as amount to send/receive command and send it

Actual behavior

Send/receive command can be used with hexadecimal number

Reproduction

  • Open Status
  • Open 1-1 or group chat
  • send to someone 0x12 ether

Additional Information

  • Status version: 0.9.8-67-g8e896fcd+
  • Operating System:
    Real device Samsung Galaxy S6, Android 6.0.1
    Real device iPhone 6s, iOS 10.2.1
@status-github-bot
Copy link

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@status-github-bot
Copy link

This issue has been automatically closed. Please re-open if this issue is important to you.

Wallet Maintenance automation moved this from Backlog/Bugs to Done Nov 28, 2018
@annadanchenko annadanchenko reopened this Apr 1, 2019
Wallet Maintenance automation moved this from Done to In progress Apr 1, 2019
@rasom rasom self-assigned this May 4, 2019
Wallet Maintenance automation moved this from In progress to Done May 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Development

Successfully merging a pull request may close this issue.

2 participants