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

Decred support #28

Closed
wants to merge 8 commits into from
Closed

Conversation

peterzen
Copy link
Contributor

Added Decred params with SegWit flag, resolved conflict

@peterzen peterzen changed the title Added Decred params, segwit flag Decred support Apr 26, 2017
Decred testnet coin type
@@ -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];
Copy link
Contributor

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,
Copy link
Contributor

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,
Copy link
Contributor

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 {
Copy link
Contributor

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,
Copy link
Contributor

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.

@peterzen
Copy link
Contributor Author

peterzen commented Jun 1, 2017

Apologies, pushed to this branch by mistake – I'll delete the PR and resubmit once complete.

@saleemrashid
Copy link
Contributor

@peterzen You don't have to delete the PR. You can force push to the branch and the PR will be updated

@prusnak prusnak force-pushed the master branch 3 times, most recently from 9777329 to 2eb9c7e Compare July 17, 2017 13:32
@prusnak prusnak force-pushed the master branch 2 times, most recently from b416c20 to 5f7a1a7 Compare August 15, 2017 15:48
@yograterol
Copy link

Any update?

@peterzen peterzen closed this Dec 11, 2017
@peterzen peterzen deleted the decred-integration branch December 11, 2017 19:53
@edwilli
Copy link

edwilli commented Jan 19, 2018

Is this a dead project :(

@knurdtech
Copy link

@edwilli no. It's on a different pull request and being reviewed for next firmware, I believe. trezor/trezor-mcu#274

matejcik pushed a commit that referenced this pull request Aug 13, 2019
- placeOrder, cancelOrder, transfer messages
- cli support
- unit and device tests
matejcik pushed a commit that referenced this pull request Nov 19, 2021
- placeOrder, cancelOrder, transfer messages
- cli support
- unit and device tests
matejcik pushed a commit that referenced this pull request Nov 22, 2021
- placeOrder, cancelOrder, transfer messages
- cli support
- unit and device tests
trezor-ci pushed a commit to vdovhanych/trezor-common that referenced this pull request Jan 26, 2022
- placeOrder, cancelOrder, transfer messages
- cli support
- unit and device tests
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.

5 participants