Skip to content

randombytes import fix#161

Closed
jcramer wants to merge 2 commits into
Bitcoin-com:stagefrom
jcramer:patch-3
Closed

randombytes import fix#161
jcramer wants to merge 2 commits into
Bitcoin-com:stagefrom
jcramer:patch-3

Conversation

@jcramer
Copy link
Copy Markdown

@jcramer jcramer commented Nov 21, 2019

Similar to the other PR, this patch fixes the following error when trying to compile ts from this source from another project.

node_modules/bitbox-sdk/lib/Mnemonic.ts:5:1
5 import * as randomBytes from "randombytes"
~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
Type originates at this import. A namespace-style import cannot be called or constructed, and will cause a failure at runtime. Consider using a default import or import require here instead.

christroutner and others added 2 commits November 19, 2019 08:01
Similar to the other PR, this patch fixes the following error when trying to compile ts from this source from another project.

 node_modules/bitbox-sdk/lib/Mnemonic.ts:5:1
    5 import * as randomBytes from "randombytes"
      ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
    Type originates at this import. A namespace-style import cannot be called or constructed, and will cause a failure at runtime. Consider using a default import or import require here instead.
@jcramer jcramer changed the base branch from master to stage November 21, 2019 15:18
@jcramer jcramer closed this Nov 21, 2019
@jcramer
Copy link
Copy Markdown
Author

jcramer commented Nov 21, 2019

I added this change to #160, so we can work with that PR.

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.

2 participants