docs: add missing await to setDoc merge example #407

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.

Sign up for GitHub

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

Jump to bottom

Open

rahman-D3V wants to merge 1 commit into firebase:master from rahman-D3V:fix-missing-await-setdoc

rahman-D3V commented Feb 7, 2026

This PR fixes a documentation inconsistency where the setDoc() example
using { merge: true } was missing the await keyword.

setDoc() is async regardless of the merge option, and other examples
on the same page already include await.

Fixes #330.


          docs: add missing await to setDoc merge example

2ea0040

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet