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

Switch @dashincubator/blocktx to dashtx & upgrade dep versions #57

Open
wants to merge 1 commit into
base: ref-blocktx
Choose a base branch
from

Conversation

jojobyte
Copy link
Contributor

This PR does a few things:

  • Upgrades all dependencies to their latest version
  • Changes all dependencies into optionalDependencies to support globalThis/window exports
    • Attempts to use Base58Check, DashTx, RIPEMD160, & Secp256k1 from globalThis/window before requiring deps
  • Adds @types/node to devDependencies
  • Upgrades @dashincubator/blocktx to dashtx@^0.16.0

dashtx@^0.18.1 is currently not used due to some breaking API changes in the package.

@jojobyte jojobyte added the enhancement New feature or request label Jul 24, 2024
@jojobyte jojobyte requested a review from coolaj86 July 24, 2024 03:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Wallet UI - 3rd Party Integration: CrowdNode.js Updates
1 participant