FirebaseExtended / firepad

Collaborative Text Editor Powered by Firebase
Other
3.87k stars 880 forks source link

firebase is not defined #363

Open toasterpic opened 4 years ago

toasterpic commented 4 years ago

Version info

Firepad: firepad@1.5.10

Firebase: firebase@7.14.1 as imported by firepad module

Ace: ng2-ace-editor@0.3.9 imports 1.4.11

Other (e.g. Node, browser, operating system) (if applicable): Angular 9.1.2 with matching angular-cli

Steps to reproduce and Test case

follow https://github.com/FirebaseExtended/firepad/issues/296

Expected behavior

Firepad will be loaded and presented

Actual behavior

Webpack warns:

WARNING in ./node_modules/firepad/dist/firepad.min.js 1:36067-36074
Critical dependency: require function is used in a way in which dependencies cannot be statically extracted

that range translates to the block starting at dist/firepad.js#L2482

and at page load time:

Uncaught ReferenceError: firebase is not defined
    at firepad.min.js:1
    at firepad.min.js:1
    at firepad.min.js:1
    at Object../node_modules/firepad/dist/firepad.min.js (firepad.min.js:1)
    at __webpack_require__ (bootstrap:84)
    at Module../src/app/firepad/firepad.component.ts (firepad.component.ts:1)
    at __webpack_require__ (bootstrap:84)
    at Module../src/app/app.module.ts (app.module.ts:1)
    at __webpack_require__ (bootstrap:84)
    at Module../src/main.ts (main.ts:1)

at Column 17956

prateekjain07 commented 2 years ago

@toasterpic - Hey, bud. Did you ever find a solution?