Skip to content

Conversation

@laanwj
Copy link
Member

@laanwj laanwj commented Jun 29, 2016

Generate an example address (with invalid CRC) for in the bitcoin address widgets, based on the network prefix, instead of hardcoding a mainnet address.

Generates:

  • 1NS17iag9jJgTHD1VXjvLCEnZuQ3rJDE9L for mainnet (same as now)
  • n2wxQmfexkjwEPgdD6iJA7T7RtzkmHxhFc for testnet

Later on we may get rid of the example address completely, but that's too late for 0.13, as removing e.g. %1 would change a translation string.

@laanwj laanwj added the GUI label Jun 29, 2016
Generate an (invalid) example address for in the bitcoin address
widgets, based on the network prefix, instead of hardcoding a mainnet
address.

- `1NS17iag9jJgTHD1VXjvLCEnZuQ3rJDE9L` for mainnet (same as now)
- `n2wxQmfexkjwEPgdD6iJA7T7RtzkmHxhFc` for testnet
@laanwj laanwj force-pushed the 2016_06_network_based_example_address branch from 5e3baf4 to 4f44cb6 Compare June 29, 2016 15:36
@maflcko
Copy link
Member

maflcko commented Jun 29, 2016

utACK 4f44cb6

@paveljanik
Copy link
Contributor

ACK 4f44cb6

@jonasschnelli
Copy link
Contributor

Yes. Why not.

utACK 4f44cb6

@maflcko maflcko added this to the 0.13.0 milestone Jun 29, 2016
@fanquake
Copy link
Member

fanquake commented Jul 5, 2016

ACK 4f44cb6

Mainnet
mainnet
Testnet
testnet

#endif
}

// Just some dummy data to generate an convincing random-looking (but consistent) address
Copy link
Member

Choose a reason for hiding this comment

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

nit s/an/a

@jonasschnelli jonasschnelli merged commit 4f44cb6 into bitcoin:master Jul 6, 2016
jonasschnelli added a commit that referenced this pull request Jul 6, 2016
4f44cb6 qt: Network-specific example address (Wladimir J. van der Laan)
codablock pushed a commit to codablock/dash that referenced this pull request Dec 28, 2017
4f44cb6 qt: Network-specific example address (Wladimir J. van der Laan)
andvgal pushed a commit to energicryptocurrency/gen2-energi that referenced this pull request Jan 6, 2019
4f44cb6 qt: Network-specific example address (Wladimir J. van der Laan)
@bitcoin bitcoin locked as resolved and limited conversation to collaborators Sep 8, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants