search

baumblatt / capacitor-firebase-auth

Capacitor Firebase Authentication Plugin
MIT License
261 stars 129 forks source link

It looks like you're using the development build of the Firebase JS SDK. #108

Closed dodomui closed 4 years ago

dodomui commented 4 years ago

I received below warning with this plugin, any advise how could I remove it? Thanks.

It looks like you're using the development build of the Firebase JS SDK.
When deploying Firebase apps to production, it is advisable to only import
the individual SDK components you intend to use.

For the module builds, these are available in the following manner
(replace <PACKAGE> with the name of a component - i.e. auth, database, etc):

CommonJS Modules:
const firebase = require('firebase/app');
require('firebase/<PACKAGE>');

ES Modules:
import firebase from 'firebase/app';
import 'firebase/<PACKAGE>';

Typescript:
import * as firebase from 'firebase/app';
import 'firebase/<PACKAGE>';
  | (anonymous) | @ | instrument.js:108
-- | -- | -- | --
  | ./node_modules/firebase/dist/index.cjs.js | @ | index.cjs.js:56
  | __webpack_require__ | @ | bootstrap:84
  | ./node_modules/capacitor-firebase-auth/dist/esm/web.js | @ | web.js:1
  | __webpack_require__ | @ | bootstrap:84
  | ./node_modules/capacitor-firebase-auth/dist/esm/facades.js | @ | facades.js:1
  | __webpack_require__ | @ | bootstrap:84
  | ./node_modules/capacitor-firebase-auth/dist/esm/index.js | @ | index.js:1
  | __webpack_require__ | @ | bootstrap:84
  | ./src/app/components/user/mobile-signup/mobile-signup.component.ts | @ | pages-user-user-logi…ogin-module.js:5606
  | __webpack_require__ | @ | bootstrap:84
  | ./src/app/pages/user/user-login/user-login.module.ts | @ | user-login-routing.module.ts:17
  | __webpack_require__ | @ | bootstrap:84
  | invoke | @ | zone-evergreen.js:364
  | onInvoke | @ | core.js:41654
  | invoke | @ | zone-evergreen.js:363
  | run | @ | zone-evergreen.js:123
  | (anonymous) | @ | zone-evergreen.js:857
  | invokeTask | @ | zone-evergreen.js:399
  | onInvokeTask | @ | core.js:41632
  | invokeTask | @ | zone-evergreen.js:398
  | runTask | @ | zone-evergreen.js:167
  | drainMicroTaskQueue | @ | zone-evergreen.js:569
  | Promise.then (async) |   |  
  | scheduleMicroTask | @ | zone-evergreen.js:552
  | scheduleTask | @ | zone-evergreen.js:388
  | onScheduleTask | @ | zone-evergreen.js:272
  | scheduleTask | @ | zone-evergreen.js:378
  | scheduleTask | @ | zone-evergreen.js:210
  | scheduleMicroTask | @ | zone-evergreen.js:230
  | scheduleResolveOrReject | @ | zone-evergreen.js:847
  | resolvePromise | @ | zone-evergreen.js:785
  | (anonymous) | @ | zone-evergreen.js:705
  | webpackJsonpCallback | @ | bootstrap:25
  | (anonymous) | @ | pages-user-user-logi…r-login-module.js:1
mesqueeb commented 4 years ago

@dodomui I think this was fixed in the latest version already. See #99 and #100

dodomui commented 4 years ago

Thanks, just realize I'm not using latest version. Solved by updated to latest version.