Skip to content
This repository has been archived by the owner on Nov 6, 2020. It is now read-only.

Token Deployment ignores 0 decimals entered #4311

Closed
jacogr opened this issue Jan 26, 2017 · 0 comments · Fixed by #4513
Closed

Token Deployment ignores 0 decimals entered #4311

jacogr opened this issue Jan 26, 2017 · 0 comments · Fixed by #4513
Assignees
@ngotchac
Labels
F2-bug 🐞 The client fails to follow expected behavior. M8-dapp 💎 Decentralized applications.
Milestone
Patch

Comments

@jacogr
Copy link
Contributor

jacogr commented Jan 26, 2017 •
edited
Loading

... while always defaulting to 6 decimals. Either -

  1. Decimals (if fixed should be explicitly stated)
  2. If decimals overrides allowed, send proper values
@jacogr jacogr added F2-bug 🐞 The client fails to follow expected behavior. M8-dapp 💎 Decentralized applications. labels Jan 26, 2017
@gavofyork gavofyork added this to the Patch milestone Jan 28, 2017
@ngotchac ngotchac self-assigned this Feb 10, 2017
ngotchac added a commit that referenced this issue Feb 10, 2017
@ngotchac
Use decimals for Token Deployment #4311
1dad307
@ngotchac ngotchac mentioned this issue Feb 10, 2017
Merged
jacogr pushed a commit that referenced this issue Feb 13, 2017
@ngotchac @jacogr
Fixes to Token Deploy dapp (#4513)
b561ae7 
* Renaming Basiccoin => Tokendeploy

* Rename Basiccoin => Tokendeploy

* UI and bug fixes to the TokenDeploy dapp

* Use decimals for Token Deployment #4311

* Typo

* PR Gumbles
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@ngotchac ngotchac

Labels
F2-bug 🐞 The client fails to follow expected behavior. M8-dapp 💎 Decentralized applications.
Projects
None yet
Milestone
Patch
Development

Successfully merging a pull request may close this issue.

Fixes to Token Deploy dapp openethereum/parity-ethereum
3 participants
@gavofyork @jacogr @ngotchac