adding missing code in a snippet

[devkiloton]

In the firebase documentation for writing and reading data there is a code snippet (the only one actually) that does not have the necessary imports. I spent a few hours on stack overflow looking for the problem because I was getting errors in my React Native app, but it was actually the documentation that did not say I had to do the imports.

The snippet is in the "Updating or deleting data" section
https://firebase.google.com/docs/database/web/read-and-write#updating_or_deleting_data

[thatfiredev]

@devkiloton Thanks for your contribution. The build is failing because you're not supposed to edit that file directly.
Instead you should update the /database-next/read-and-write.js file. I can see in the writeNewPost_wrapped() function that the imports are already there, they just happen to be outside of the snippet comments (// [START rtdb_social_write_fan_out]).

So the best solution here would be to move that snippet comment to the start of the function so that the imports are shown:

function writeNewPost_wrapped() {
  // [START rtdb_social_write_fan_out]
  const { getDatabase, ref, child, push, update } = require("firebase/database");

[thatfiredev]

@devkiloton Please also undo the change that you made to the generated file.

[devkiloton]

@devkiloton Thanks for your contribution. The build is failing because you're not supposed to edit that file directly. Instead you should update the /database-next/read-and-write.js file. I can see in the writeNewPost_wrapped() function that the imports are already there, they just happen to be outside of the snippet comments (// [START rtdb_social_write_fan_out]).

So the best solution here would be to move that snippet comment to the start of the function so that the imports are shown:

function writeNewPost_wrapped() {
  // [START rtdb_social_write_fan_out]
  const { getDatabase, ref, child, push, update } = require("firebase/database");

Thanks @thatfiredev! I can't believe that the answer for the problem was literally in my screen.

[thatfiredev]

LGTM

[devkiloton]

LGTM

@thatfiredev should not I create a commit with the generated file? "npm run snippets" will do that.

[thatfiredev]

@devkiloton Oops, my bad! Seems like you're right, please do commit the generated file.

[devkiloton]

Thanks for the support @thatfiredev!

[thatfiredev]

@devkiloton the change is now live in the docs. 🎉

[devkiloton]

@devkiloton the change is now live in the docs. 🎉

@thatfiredev dude, this is insane! Firebase has a piece of me now haha! Thanks for the support again!