-
-
Notifications
You must be signed in to change notification settings - Fork 233
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
Decred support #28
Decred support #28
Conversation
Decred testnet coin type
257eebf
to
0316f61
Compare
@@ -81,7 +81,10 @@ enum MessageType { | |||
MessageType_DebugLinkMemoryRead = 110 [(wire_debug_in) = true]; | |||
MessageType_DebugLinkMemory = 111 [(wire_debug_out) = true]; | |||
MessageType_DebugLinkMemoryWrite = 112 [(wire_debug_in) = true]; | |||
MessageType_DebugLinkFlashErase = 113 [(wire_debug_in) = true]; | |||
MessageType_DebugLinkFlashErase = 113 [(wire_debug_in) = true]; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happened here?
"default_fee_b": { | ||
"Normal": 10 | ||
}, | ||
"dust_limit": 546, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
What happened here?
"default_fee_b": { | ||
"Normal": 10 | ||
}, | ||
"dust_limit": 546, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Indentation, again.
* @next Success | ||
* @next Failure | ||
*/ | ||
message DecredLoadDevice { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you need a new message or could you have added it as an enum in the existing message?
"hash_genesis_block": "298e5cc3d985bfe7f81dc135f360abe089edd4396b86d2de66b0cef42b21d980", | ||
"xpub_magic": "043587d1", | ||
"xprv_magic": "04358397", | ||
"bip44": 11, |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Testnet should use 1
. 11
is registered to another coin.
Apologies, pushed to this branch by mistake – I'll delete the PR and resubmit once complete. |
@peterzen You don't have to delete the PR. You can force push to the branch and the PR will be updated |
9777329
to
2eb9c7e
Compare
b416c20
to
5f7a1a7
Compare
Any update? |
Is this a dead project :( |
@edwilli no. It's on a different pull request and being reviewed for next firmware, I believe. trezor/trezor-mcu#274 |
- placeOrder, cancelOrder, transfer messages - cli support - unit and device tests
- placeOrder, cancelOrder, transfer messages - cli support - unit and device tests
- placeOrder, cancelOrder, transfer messages - cli support - unit and device tests
- placeOrder, cancelOrder, transfer messages - cli support - unit and device tests
Added Decred params with SegWit flag, resolved conflict