snippets-web icon indicating copy to clipboard operation
snippets-web copied to clipboard

missing keyword `await` for the set_with_merge.js file

Open waldothedeveloper opened this issue 1 year ago • 1 comments

The docs related to Set a document see here are missing the keyword await before setDoc() function. For the code example where the { merge: true} is used.

image

All of the previous and subsequent code snippets do contain the word await for the setDoc() function. Does the {merge: true} change the async behavior of the setDoc() function?

If not, the await should be present in that snippet to match all of the other code snippets for this page.

Here's another example on the same page where the await keyword is present for the setDoc() function:

image

waldothedeveloper avatar Mar 03 '23 16:03 waldothedeveloper

I can submit a PR if needed :)

waldothedeveloper avatar Mar 03 '23 16:03 waldothedeveloper