Skip to content

Conversation

@dcousens
Copy link
Contributor

Targeted for 2.0.0.

This is a continued discussion of #306 which was closed due to the 3.0.0 branch being deleted.

@coveralls
Copy link

Coverage Status

Coverage decreased (-0.0%) when pulling 597fd2e on noaddr into cbb9b8e on 2.0.0.

@dcousens
Copy link
Contributor Author

Thoughts were to do address.classifyNetwork (or at least, findNetwork) and getVersion.
I'm not sure if there was consensus on whether decode/encode should remain at all.

@dcousens dcousens added this to the 2.0.0 milestone Nov 26, 2014
@weilu
Copy link
Contributor

weilu commented Nov 27, 2014

I thought the point of having classifyNetwork and getVersion is so that we can not expose encode/decode

@dcousens dcousens force-pushed the 2.0.0 branch 6 times, most recently from f1e8442 to 2746027 Compare November 28, 2014 03:05
@dcousens
Copy link
Contributor Author

@weilu would classifyNetwork work on base58 or the version number? Otherwise we're doing a base58check.decode twice?

edit: Though I guess you're probably only ever going to need one or the other.

@dcousens dcousens force-pushed the 2.0.0 branch 2 times, most recently from 093e8f8 to c536926 Compare November 29, 2014 01:37
@weilu
Copy link
Contributor

weilu commented Nov 29, 2014

Yeah, base58 was my thought

@dcousens
Copy link
Contributor Author

Ok, will rebase, update then notify

dcousens and others added 24 commits March 19, 2015 13:29
crypto: use packages directly
add a note about uglying (to avoid issues with typeForce)
bufferutils: remove equal, use Buffer.compare
Add Example on Generating a Random Address and WIF for Alternative Network
@dcousens
Copy link
Contributor Author

Will re-open as this now meant to be merged into master.

@dcousens dcousens restored the noaddr branch April 28, 2015 02:06
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.

7 participants