Log correct version for v9 Firestore and RTDB

[Feiyang1]

No description provided.

[changeset-bot]

⚠️ No Changeset found

Latest commit: 646df6c

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[google-oss-bot]

Binary Size Report

Affected SDKs

• @firebase/database-exp

  Type	Base (919413c)	Head (99bff18)	Diff
  browser	245 kB	245 kB	+110 B (+0.0%)
  main	277 kB	277 kB	+83 B (+0.0%)
  module	245 kB	245 kB	+110 B (+0.0%)

• @firebase/firestore

  Type	Base (919413c)	Head (99bff18)	Diff
  browser	283 kB	283 kB	+4 B (+0.0%)
  main	529 kB	529 kB	+44 B (+0.0%)
  module	283 kB	283 kB	+4 B (+0.0%)

• @firebase/firestore-compat

  Type	Base (919413c)	Head (99bff18)	Diff
  browser	28.4 kB	28.5 kB	+120 B (+0.4%)
  main	37.4 kB	37.5 kB	+120 B (+0.3%)
  module	28.4 kB	28.5 kB	+120 B (+0.4%)
  react-native	28.1 kB	28.3 kB	+120 B (+0.4%)

• @firebase/firestore-exp

  Type	Base (919413c)	Head (99bff18)	Diff
  browser	223 kB	223 kB	+76 B (+0.0%)
  main	505 kB	505 kB	+119 B (+0.0%)
  module	223 kB	223 kB	+76 B (+0.0%)
  react-native	223 kB	223 kB	+64 B (+0.0%)

• @firebase/firestore-lite

  Type	Base (919413c)	Head (99bff18)	Diff
  browser	71.4 kB	71.5 kB	+83 B (+0.1%)
  main	145 kB	145 kB	+127 B (+0.1%)
  module	71.4 kB	71.5 kB	+83 B (+0.1%)
  react-native	71.6 kB	71.7 kB	+74 B (+0.1%)

• @firebase/firestore/bundle

  Type	Base (919413c)	Head (99bff18)	Diff
  main	525 kB	525 kB	+44 B (+0.0%)

• @firebase/firestore/memory

  Type	Base (919413c)	Head (99bff18)	Diff
  main	323 kB	324 kB	+44 B (+0.0%)

• @firebase/firestore/memory-bundle

  Type	Base (919413c)	Head (99bff18)	Diff
  main	320 kB	320 kB	+44 B (+0.0%)

Test Logs

• Base (919413cd): https://github.com/firebase/firebase-js-sdk/actions/runs/790799128
• Head (99bff18c): https://github.com/firebase/firebase-js-sdk/actions/runs/793435023

[google-oss-bot]

Size Analysis Report

Affected Products

Diffs between base commit (919413c) and head commit (94ffdd8) are too large (479,509 characters) to display.

Please check below links to see details from the original test log.

Test Logs

• Head (94ffdd8): https://github.com/firebase/firebase-js-sdk/actions/runs/791128341

[schmidt-sebastian]

Can we differentiate Firestore lite from Firestore? I wonder if we should prefix or postfix this with lite.

[Feiyang1]

like 9.0.0-beta.1_lite?

[hsubox76]

Don't we normally put variant info in the product string and not in the version number string? It's already in the product string.

[hsubox76]

The product string is in line 55, I think that should be enough to distinguish for platform logging, unless this is for Firestore-only backend logging? I think it might make platform logging queries a little harder to have one version not like the others.

[Feiyang1]

This version is logged to Firestore backend directly. We will have to do some data massage to join Firestore data with platform logging data.

[schmidt-sebastian]

Please fix lint.

[Feiyang1]

I swear I ran lint locally and everything passed!