-
Notifications
You must be signed in to change notification settings - Fork 41
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 new @hemilabs/pop-miner NPM package #162
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
All looks reasonable to me.
OK
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We discussed a couple of things on slack, like how the front-end would provide the scriptHash to call the bitcoin balance or utxos functions, or when the user input a bitcoin private key only, how the front-end would know all the other info that is provided by generating a new key, or how the front-end will render the transactions list table, where the user will see the latests blocks and its status.
These are topics for future PR's, so LGTM.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM - I just left a couple of minor suggestions / improvements 😄
#162 (comment) Suggested-by: Gonzalo D'Elia <gonzalo@bloq.com>
Add the initial implementation of the `@hemilabs/pop-miner` NPM package, a TypeScript API layer for the WebAssembly PoP Miner. Also adds supporting files, GitHub Actions workflow changes to build and publish the NPM package. This is the initial version of this package, more functionality will be added in future pull requests. Reviewed-on: hemilabs#162
Summary
Add
@hemilabs/pop-miner
package, a TypeScript API for the WebAssembly PoP Miner.Changes
@hemilabs/pop-miner
package (web/packages/pop-miner
) and supporting filesThis is the initial version of this package, more functionality will be added in future pull requests.
This contains more than +9K line changes, however the majority of this is from the PNPM lock files and can be ignored.