FR: add ability to set `ignoreUndefinedProperties` after `getFirestore`

[jvliwanag]

Operating System

n/a

Environment (if applicable)

n/a

Firebase SDK Version

11.4.0

Firebase SDK Product(s)

Firestore

Project Tooling

n/a

Detailed Problem Description

Currently, the only way to set ignoreUndefinedProperties using the modular api is to use initializeFirestore, which has different semantics as getFirestore.

While firestore-compat has a way around this, it seems to be wrapped in private api's. Looking for an api on firestore to set this directly.

Steps and code to reproduce issue

n/a

[dconeybe]

@jvliwanag Could you suggest some ideas of what the proposed API would look like? Also, which private API are you referring to in the firestore-compat library?

[jvliwanag]

I imagine we can either do:

• setSettings(firestore: Firestore, settings: FirestoreSettings) similar to how compat works
• OR setignoreUndefinedProperties if the other settings might not be applicable.

Note that on firestore-compat, we allow setting these here. However this calls this._delegate._setSettings which I don't think is an exposed api.

[dconeybe]

Got it. Another question: why is it problematic/troublesome for you to use initializeFirestore()?

[jvliwanag]

While initializeFirestore works, I prefer to use the conveniences of getFirestore including auto detecting emulator settings from environment rather than recreating it myself.

Moreover, I prefer to use the get pattern across firebase services such as Auth. Auth for instance allows me to change some settings after getting it such as in setPersistence.

[dconeybe]

Thanks for this information. I'll bring your request to the team and will respond in about a week or so.