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

Added DeepBrain Coin #448

Merged
merged 1 commit into from
Jan 1, 2018
Merged

Added DeepBrain Coin #448

merged 1 commit into from
Jan 1, 2018

Conversation

mhuggins
Copy link
Contributor

What current issue(s) from Trello/Github does this address?
N/A

What problem does this PR solve?
It supports sending & showing balance for DBC (DeepBrain Coin), similar to RPX.

How did you solve this problem?
I added the MainNet & TestNet constants for the DBC hash.

How did you make sure your solution works?
I loaded the wallet and confirmed that I can view the 0 DBC balance & view DBC token information by clicking on the balance. NOTE: I was unable to test transferring DBC due to having a 0 balance.

Are there any special changes in the code that we should be aware of?
N/A

Is there anything else we should know?
See NOTE above.

  • Unit tests written?

@mhuggins
Copy link
Contributor Author

The test failure doesn't appear to relate to my change.

@slipo
Copy link
Contributor

slipo commented Dec 31, 2017

Just fyi: I had the same issue with the build. Here was my fix: bfe2090

@coveralls
Copy link

Coverage Status

Coverage remained the same at 41.246% when pulling 005495c on neoverse:dbc-token into 7012097 on CityOfZion:dev.

@mhuggins
Copy link
Contributor Author

mhuggins commented Jan 1, 2018

@slipo Thanks, rebasing against dev fixed it!

@evgenyboxer evgenyboxer merged commit cb25830 into CityOfZion:dev Jan 1, 2018
@ministoat
Copy link

Using a ledger I cannot send dbc with this version, error message says not supported

@mhuggins
Copy link
Contributor Author

mhuggins commented Jan 3, 2018

@ministoat I don't have a ledger to test against. I will have to see if someone else can help out.

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

Successfully merging this pull request may close these issues.

None yet

5 participants