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

engine: build the UMD module with a globalObject of this #282

Merged
merged 1 commit into from
Nov 6, 2023

Conversation

pkgw
Copy link
Contributor

@pkgw pkgw commented Nov 6, 2023

The Webpack default is self but as far as I can tell, this is preferable:

https://webpack.js.org/configuration/output/#outputglobalobject
https://stackoverflow.com/questions/64639839/typescript-webpack-library-generates-referenceerror-self-is-not-defined

This fixes a problem with testing/running the Constellations frontend with the latest version of the engine.

The Webpack default is `self` but as far as I can tell, `this` is
preferable:

https://webpack.js.org/configuration/output/#outputglobalobject
https://stackoverflow.com/questions/64639839/typescript-webpack-library-generates-referenceerror-self-is-not-defined

This fixes a problem with testing/running the Constellations frontend
with the latest version of the engine.
@pkgw pkgw merged commit 0f7783c into WorldWideTelescope:master Nov 6, 2023
9 checks passed
@pkgw pkgw deleted the globalobject-this branch November 6, 2023 20:54
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.

1 participant