Skip to content
This repository was archived by the owner on Aug 30, 2022. It is now read-only.

Conversation

jameshowe
Copy link
Contributor

@jameshowe jameshowe commented Jun 2, 2022

When attempting to load the SDK from a Node script the Browser package is being loaded. This is because module does not appear to be a known condition and therefore it skips and loads the default entrypoint (browser).

When attempting to load the SDK from a Node script the Browser package is being loaded. This is because `module` does not appear to be a [known condition](https://nodejs.org/api/packages.html#conditional-exports) and therefore it skips and ends up loading the `default` ESModule.
@jameshowe jameshowe changed the title Bugfix - Browser package being loaded when using ESModules Bugfix - Incorrect module being loaded when using Node ESModules Jun 2, 2022
@jameshowe jameshowe changed the title Bugfix - Incorrect module being loaded when using Node ESModules Bugfix - Incorrect entrypoint being loaded when using Node ESModules Jun 2, 2022
Copy link
Member

@jnsdls jnsdls left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

please keep module around in addition to import

thanks!

Include 'module'

Co-authored-by: Jonas Daniels <jonas.daniels@outlook.com>
@jnsdls
Copy link
Member

jnsdls commented Jun 2, 2022

thanks @jameshowe!

@jnsdls jnsdls merged commit 9156a55 into thirdweb-dev:main Jun 2, 2022
@jameshowe jameshowe deleted the patch-1 branch June 2, 2022 19:12
@jameshowe
Copy link
Contributor Author

thanks @jameshowe!

No worries! Any view on when this will be released? (I can patch my local version just now)

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants