Use smaller Firestore imports

Oh goodie, the same error message (Uncaught SyntaxError: The requested module './../../../../../app/dist/index.esm.js' does not provide an export named 'SDK_VERSION' (at index.esm2017.js:1:93)) is showing up even after moving to rollup build...

jkomoros commented 2 years ago

There's likely something going on about bare module specifiers and the way they're handled in both lit / rollup : https://lit.dev/docs/v1/tools/build/#bare-module-specifiers, leading to the bare module specifiers in the firebase packages being rewritten to incorrect versions of themselves somehow?

jkomoros commented 2 years ago

https://firebase.google.com/docs/web/module-bundling

jkomoros commented 2 years ago

[x] Use firebase 9 in cloud functions
[x] make sure tests work
[x] Make sure builds work
[x] Figure out how to re-enable persistence (see what happens in terms of timing of changes in cache, etc)
[x] Verify screenshot service works (make sure to deploy updated client)
[x] Run npm install --save firebase-functions@latest in functions folder
[x] Make sure maintenance page works
[x] Make sure permissions page works
[x] Manually eyeball every change in 7260a3127cc45162a56aa94bc85364dd041803d3 to verify nothing looks out of place
[x] Actually get it working
[x] Fix MultiBatch sentinel
[x] Get rid of the indirect deleteSentinel, serverTimestampSentinel in a few packages

jkomoros commented 2 years ago

Being developed in firebase-modular branch

jkomoros commented 2 years ago

Tests are having the compat import problem. https://github.com/firebase/firebase-js-sdk/issues/5687 seems vaguely related?