web3-onboard icon indicating copy to clipboard operation
web3-onboard copied to clipboard

[Bug]: eip-712 peer dependency types are missing

Open enzoferey opened this issue 3 years ago • 3 comments

Current Behavior

When using any package that uses web3-onboard/common in a TypeScript project, if I run tsc --noEmit with skipLibCheck: false I'm getting:

node_modules/@web3-onboard/common/dist/types.d.ts:4:51 - error TS2307: Cannot find module 'eip-712' or its corresponding type declarations.

export type { TypedData as EIP712TypedData } from 'eip-712';

Expected Behavior

I expect the packages to bundle their internal required types.

Steps To Reproduce

  1. Install any package that uses web3-onboard/common.
  2. Set skipLibCheck: false in your tsconfig.json file.
  3. Run tsc --noEmit.

Onboard Version

v2

Node Version

v16.14.0

What browsers are you seeing the problem on?

No response

Relevant log output

node_modules/@web3-onboard/common/dist/types.d.ts:4:51 - error TS2307: Cannot find module 'eip-712' or its corresponding type declarations.

export type { TypedData as EIP712TypedData } from 'eip-712';

Anything else?

No response

enzoferey avatar Apr 27 '22 18:04 enzoferey

@enzoferey are you still experiencing this issue I think it is fixed now at least on my end.

taylorjdawson avatar May 23 '22 22:05 taylorjdawson

Hey @taylorjdawson ! Yes, it still happens when skipLibCheck: false. I just noticed I forgot to mention it, sorry for that, I'm updating the issue description.

enzoferey avatar May 24 '22 08:05 enzoferey

@enzoferey Cool! If you want to PR to fix that would be great! Otherwise we'll get to this later as it's a lower priority for now 👍🏾

taylorjdawson avatar May 25 '22 00:05 taylorjdawson

@enzoferey is the problem persisting? I believe it should have been fixed in the latest package versions. Thanks!

Adamj1232 avatar Dec 15 '22 18:12 Adamj1232

Hey @Adamj1232 unfortunately I'm no longer working on this project, so I cannot double check. If you believe it should be solved now please go ahead and just close the issue, I will reopen if it bites me again in the future 😄

enzoferey avatar Dec 15 '22 21:12 enzoferey