cssinjs/jss

UMD builds not working on import

Open

#1.567 geöffnet am 21. Okt. 2021

Auf GitHub ansehen
 (6 Kommentare) (0 Reaktionen) (0 zugewiesene Personen)JavaScript (7.000 Stars) (434 Forks)batch import
bugcomplexity:lowhelp wanted

Beschreibung

Expected behavior: I'm trying to implement react-jss via CDN. Importing the UMD build via <script> tag should work out of the box.

Describe the bug: The module does not load and the console generates the error as follows:

Uncaught ReferenceError: module is not defined
    at react-jss.min.js:6
    at react-jss.min.js:1
    at react-jss.min.js:1

Reproduction:

CodeSandbox - loading react, react-dom, and react-jss

Versions:

  • jss: 10.8.1
  • browser: any

Contributor Guide

Tech Stack
javascriptreact
Domain
frontendbuild system
Issue Type
bug
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
half day
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
fresh
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
mostly clear
Voraussetzungen
Understanding of UMD module formatBasic bundler configuration knowledge
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
60
Research-Richtung
The issue is that the UMD bundle for react jss references `module` which is not defined in the browser. Investigate the Rollup configuration (likely `rollup.config.js` or `package.json` build scripts) to see how the UMD output is generated. Look for any missing `output.globals` or improper use of CommonJS. The fix may involve ensuring the bundle does not depend on `module` or adding a shim. Check the existing build setup and compare with other packages that successfully produce UMD bundles.