-
Notifications
You must be signed in to change notification settings - Fork 929
Partial imports with rollup failing #47
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
Comments
Hey there! I couldn't figure out what this issue is about, so I've labeled it for a human to triage. Hang tight. |
FYI, it works fine if you change your imports to: import * as firebase from 'firebase/app';
import 'firebase/database'; |
Hi @jsayol. Could you clarify what rollup plugins and/or settings you are using? If used directly I get Here is my environment:
|
@jshcrowthe , can you showcase firebase how to bind rollupjs? |
@jshcrowthe there is an issue filed on the rollup repo, linking a MCVE to reproduce the underlying issue. |
Closing this due to inactivity and since it seems to be an issue w/ the rollup lib and not Firebase |
Following up on #45 and on @jshcrowthe's comment:
Use case: just importing the database layer to develop and unit test individual modules that need a common async data store. They will likely eventually be used in the browser but development, unit testing, and packaging is just so much easier and faster in node.
>rollup -o app.bld.js -f cjs app.js
Tried importing from ./node_modules/firebase/firebase-app but being build for the browser I get a navigator error in node
I also tried many a few combinations directly in the firebase/app/ folder with browserify to get an intermediate build that would be easier to consume but the build system looks like it was hacked to suit babel.
I understand I am trying to use the database component in a way that is not the intended typical use but I thought that if there were less frictions in just using the firebase toolset it would be much easier to get there progressively.
The text was updated successfully, but these errors were encountered: