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 bitcoin testnet testnet.blockexplorer.com #133

Open
wants to merge 10 commits into
base: master
Choose a base branch
from

Conversation

roundminer
Copy link

I have added testnet https://testnet.blockexplorer.com for retrieving unspent data, broadcasting a transaction to testnet blockchain and generating addresses.

In the settings page on Broadcast tab, there is now an option for Bitcoin testnet which populates "Broadcast" and "Unspent outputs" dropdown menus. Here is how it looks:
net-broadcast-unspent-setting

On New transaction tab I have made extra recipient div with autopopulated address field taken from sender address and set it as readonly (greyed-out). Amount is left like for all other recipients so user must input remaning amount in it. Calculation of transaction fees remained the same because basicaly this is just another recipient added to transaction. Here is how it looks:
new-tx-with-return-address

Finally, broadcasting the transaction to testnet had a link added to https://testnet.blockexplorer.com, so users could check the transaction on testnet explorer. Had to do it this way 'cause direct link to transaction on testnet.blockexplorer.com doesn't work. Here is how it looks when transaction is broadcasted:
rawsubmit-broadcast-testnet-tx

Donation addresses:
BTC: 1DGX4Zu83i3zL3zCNRaiat6tbtPMgEyhNE
BCH: 17eejjY1kBTGo79grQaJR8NkvoujX9r8ce
ETH: 0xA9b7226D8a0Daf91F614c5E7511C3E10d6e0B10c
LTC: LMYWNKfEjqAj3NFG4Wcj3oSpYwvbRSbyCy

@nickfogle
Copy link

Thanks @roundminer! I can confirm that testnet functionality is working after these changes.

@Onefox
Copy link

Onefox commented Apr 6, 2018

Thanks for your work @roundminer! I am just trying out the functionallity.
If you are on the test net the following link doesn't work anymore:
image

Copy link
Contributor

@yottalogical yottalogical left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Make sure to update sha1sum, especially for files without conflicts.

Just run shasum FILENAME from the terminal, then copy the new digests into the file sha1sum.

@roundminer roundminer closed this Apr 14, 2019
@roundminer roundminer deleted the add-testnet-blockexplorer branch April 14, 2019 11:13
@roundminer roundminer restored the add-testnet-blockexplorer branch April 14, 2019 11:13
@roundminer roundminer reopened this Apr 14, 2019
Add Link to Blockchain Explorer after Broadcasting Transaction
@roundminer
Copy link
Author

@yottalogical
I messed something up with my code, tried to fix it and as it seems I succeded in fixing the prob, as now I see it says the branch has no conflicts, so if you could please check if this is ok to merge, tell me what else to change or make pull requests to my branch if necessary.
At first I wasn't able to merge your pull request (roundminer#1), but after I made a revert of my messup, I was able.. Something odd happened at my repo, sry for incovenience.

@yottalogical
Copy link
Contributor

Everything appears to work fine! I tested Bitcoin Testnet functionality, but also tested a different currency (in this case, dogecoin) to make sure nothing else was messed up.

@smartm0use
Copy link

Hello @roundminer, sorry for resuming this old post. Are you able to restore the testnet support to Coinbin wallet?

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