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

popm/wasm: add 'parseKey' method #168

Merged
merged 1 commit into from
Jul 4, 2024
Merged

Conversation

joshuasing
Copy link
Contributor

@joshuasing joshuasing commented Jul 3, 2024

Summary
Add a parseKey method to the WebAssembly PoP Miner and @hemilabs/pop-miner package that takes a network type and secp256k1 private key and returns the public key, Bitcoin address and public key hash, and the Ethereum address.

Changes

  • Add parseKey method to the WebAssembly PoP Miner.
  • Rename GenerateKeyResult to KeyResult in pop-miner package.
  • Fix typo secpk256k1 -> secp256k1.

@joshuasing joshuasing added type: refactor This refactors existing functionality type: enhancement This improves existing functionality size: M This change is medium (+/- <200) labels Jul 3, 2024
@joshuasing joshuasing requested a review from a team as a code owner July 3, 2024 20:18
@github-actions github-actions bot added area: popm/wasm This is a change to the WASM popm (PoP Miner) area: @hemilabs/pop-miner This is a change to the @hemilabs/pop-miner NPM package labels Jul 3, 2024
Copy link
Contributor

@jcvernaleo jcvernaleo left a comment

Choose a reason for hiding this comment

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

OK

Copy link

@ArturDolzan ArturDolzan left a comment

Choose a reason for hiding this comment

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

LGTM

@joshuasing joshuasing merged commit fe9752a into main Jul 4, 2024
6 checks passed
@joshuasing joshuasing deleted the joshua/popm/wasm-bitcoinKey branch July 4, 2024 17:19
@joshuasing joshuasing added type: feature This adds new functionality and removed type: enhancement This improves existing functionality labels Jul 5, 2024
web3cryptoguy pushed a commit to web3cryptoguy/heminetwork that referenced this pull request Nov 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area: @hemilabs/pop-miner This is a change to the @hemilabs/pop-miner NPM package area: popm/wasm This is a change to the WASM popm (PoP Miner) size: M This change is medium (+/- <200) type: feature This adds new functionality type: refactor This refactors existing functionality
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants