Skip to content

In README, links to Bitcoin testnet explorer were updated - #112

Open
imestin wants to merge 1 commit into
decred:masterfrom
imestin:master
Open

In README, links to Bitcoin testnet explorer were updated#112
imestin wants to merge 1 commit into
decred:masterfrom
imestin:master

Conversation

@imestin

@imestin imestin commented Jul 11, 2019

Copy link
Copy Markdown

The links for Decred (testnet) are still not working, because non of the block explorers I know about could find those transactions.

The Bitcoin links were not working, because blocktrail.com ceased operations.

@jrick

jrick commented Jul 11, 2019

Copy link
Copy Markdown
Member

The examples were done on testnet2 and decred is currently on testnet3, so they need to be entirely redone.

@imestin

imestin commented Jul 12, 2019

Copy link
Copy Markdown
Author

I tried to recreate the transactions.

After I run ./bitcoin-cli -testnet getnewaddress

./btcatomicswap --testnet --rpcuser=user --rpcpass=AtomicSwap1234 initiate 2N8S4gAgJSY8G9fMmRoR2hoNprgRFWRFcT2 0.001
will result:
participant address is not P2PKH

@imestin

imestin commented Jul 13, 2019

Copy link
Copy Markdown
Author

I tried to create a SegWit compatible address in Bitcoin-qt, that also didn't work.
tb1q3nw54nyj44eewnkswv2jtqhqrl7rpr2t7tqmm5

I'm still trying to figure this out, if I will have a solution I will post it here.

@jrick

jrick commented Jul 15, 2019

Copy link
Copy Markdown
Member

You need to generate p2pkh addresses (aka legacy). getnewaddress should let you specify the kind.

@imestin

imestin commented Jul 16, 2019

Copy link
Copy Markdown
Author

OK, it works.

bitcoin-cli -testnet getnewaddress random_comment legacy
will work

@imestin

imestin commented Jul 16, 2019

Copy link
Copy Markdown
Author

signrawtransaction: -32: signrawtransaction was removed in v0.18. Clients should transition to using signrawtransactionwithkey and signrawtransactionwithwallet

@imestin

imestin commented Jul 16, 2019

Copy link
Copy Markdown
Author

I will create an issue with the above mentioned problem.

UPDATE: there is already a pull request for that.
#109

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.

2 participants