[FR] Show warning if passing undefined value to Firestore's update()

[risalfajar]

Is your feature request related to a problem? Please describe.
I'm always frustrated when passing an undefined value to update() call without any issue in the IDE/lint, but then getting a runtime error after the function is deployed.

Describe the solution you'd like
Show warning in IDE if an undefined value is passed to update()

const value: string | undefined = undefined

admin.firestore(app)
    .collection("collection")
    .doc("docId")
    .update({
        id: value // show a warning here
    })

[google-oss-bot]

I found a few problems with this issue:

• I couldn't figure out how to label this issue, so I've labeled it for a human to triage. Hang tight.
• This issue does not seem to follow the issue template. Make sure you provide all the required information.

[cherylEnkidu]

Hi @risalfajar,
Given that in some scenarios undefined is a valid use case, we cannot restrict the type here. If it is helpful in your case, consider setting ignoreUndefinedProperties?: boolean; to true in https://googleapis.dev/nodejs/firestore/latest/Firestore.html#settings

[cherylEnkidu]

Hi,

I will close this ticket for now, please feel free to reopen it if you have more concerns.