Skip to content

Conversation

ziad-saab
Copy link
Contributor

Declare this dependency explicitly, as it's currently necessary for the JSX components to work proplerly.

The same deps are already in the site package and get hoisted, which unintendedly makes everything work.

Declare this dependency explicitly, as it's currently necessary
for the JSX components to work proplerly.

The same deps are already in the site package and get hoisted, which
unintendedly makes everything work.
@ziad-saab ziad-saab requested a review from a team as a code owner October 21, 2024 19:14
Copy link

socket-security bot commented Oct 21, 2024

New dependencies detected. Learn more about Socket for GitHub ↗︎

Package New capabilities Transitives Size Publisher
npm/@types/react-dom@18.2.4 None 0 30.3 kB types
npm/@types/react@18.2.4 None +3 1.61 MB types

View full report↗︎

@Mrtenz
Copy link
Member

Mrtenz commented Oct 21, 2024

Maybe we should add these to @metamask/snaps-sdk until we can fix the underlying issue?

@FrederikBolding
Copy link
Member

Maybe we should add these to @metamask/snaps-sdk until we can fix the underlying issue?

Maybe, but this is a good stopgap solution as well

@FrederikBolding FrederikBolding merged commit 03b5556 into main Oct 22, 2024
11 checks passed
@FrederikBolding FrederikBolding deleted the zs/add-react-types branch October 22, 2024 08:59
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.

3 participants