Cloud Firestore Function returns only one document?











up vote
0
down vote

favorite












I create a firebase function to return posts collection, for some reason cloud function return only one post(document). But when I run this function locally, in terminal I get all posts. Any idea why?



const functions = require('firebase-functions');
const admin = require('firebase-admin');
const cors = require('cors')({origin: true});
admin.initializeApp();

exports.postWidget = functions.https.onRequest((request, response) => {
cors(request, response, () => {});

var db = admin.firestore();

var postsRef = db.collection('posts')
postsRef.where('postStatus', '==', 'published').get()
.then(snapshot => {
snapshot.forEach(doc => {
let data = doc.data()
return response.send(data)
});
})
.catch(err => {
console.log('Error getting documents', err);
});

});









share|improve this question




























    up vote
    0
    down vote

    favorite












    I create a firebase function to return posts collection, for some reason cloud function return only one post(document). But when I run this function locally, in terminal I get all posts. Any idea why?



    const functions = require('firebase-functions');
    const admin = require('firebase-admin');
    const cors = require('cors')({origin: true});
    admin.initializeApp();

    exports.postWidget = functions.https.onRequest((request, response) => {
    cors(request, response, () => {});

    var db = admin.firestore();

    var postsRef = db.collection('posts')
    postsRef.where('postStatus', '==', 'published').get()
    .then(snapshot => {
    snapshot.forEach(doc => {
    let data = doc.data()
    return response.send(data)
    });
    })
    .catch(err => {
    console.log('Error getting documents', err);
    });

    });









    share|improve this question


























      up vote
      0
      down vote

      favorite









      up vote
      0
      down vote

      favorite











      I create a firebase function to return posts collection, for some reason cloud function return only one post(document). But when I run this function locally, in terminal I get all posts. Any idea why?



      const functions = require('firebase-functions');
      const admin = require('firebase-admin');
      const cors = require('cors')({origin: true});
      admin.initializeApp();

      exports.postWidget = functions.https.onRequest((request, response) => {
      cors(request, response, () => {});

      var db = admin.firestore();

      var postsRef = db.collection('posts')
      postsRef.where('postStatus', '==', 'published').get()
      .then(snapshot => {
      snapshot.forEach(doc => {
      let data = doc.data()
      return response.send(data)
      });
      })
      .catch(err => {
      console.log('Error getting documents', err);
      });

      });









      share|improve this question















      I create a firebase function to return posts collection, for some reason cloud function return only one post(document). But when I run this function locally, in terminal I get all posts. Any idea why?



      const functions = require('firebase-functions');
      const admin = require('firebase-admin');
      const cors = require('cors')({origin: true});
      admin.initializeApp();

      exports.postWidget = functions.https.onRequest((request, response) => {
      cors(request, response, () => {});

      var db = admin.firestore();

      var postsRef = db.collection('posts')
      postsRef.where('postStatus', '==', 'published').get()
      .then(snapshot => {
      snapshot.forEach(doc => {
      let data = doc.data()
      return response.send(data)
      });
      })
      .catch(err => {
      console.log('Error getting documents', err);
      });

      });






      firebase google-cloud-firestore google-cloud-functions






      share|improve this question















      share|improve this question













      share|improve this question




      share|improve this question








      edited Nov 13 at 8:24

























      asked Nov 12 at 6:09









      SinishaS

      34




      34
























          1 Answer
          1






          active

          oldest

          votes

















          up vote
          0
          down vote



          accepted










          You're calling response.send(data) multiple times in forEach loop. That's not likely to do what you want, as you're only supposed to call send() once with then entire response. Instead, you should be collecting all the data to send in a single object, and call send() with that one object.






          share|improve this answer





















          • Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
            – SinishaS
            Nov 12 at 6:24













          Your Answer






          StackExchange.ifUsing("editor", function () {
          StackExchange.using("externalEditor", function () {
          StackExchange.using("snippets", function () {
          StackExchange.snippets.init();
          });
          });
          }, "code-snippets");

          StackExchange.ready(function() {
          var channelOptions = {
          tags: "".split(" "),
          id: "1"
          };
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function() {
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled) {
          StackExchange.using("snippets", function() {
          createEditor();
          });
          }
          else {
          createEditor();
          }
          });

          function createEditor() {
          StackExchange.prepareEditor({
          heartbeatType: 'answer',
          convertImagesToLinks: true,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: 10,
          bindNavPrevention: true,
          postfix: "",
          imageUploader: {
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          },
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          });


          }
          });














          draft saved

          draft discarded


















          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53256695%2fcloud-firestore-function-returns-only-one-document%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown

























          1 Answer
          1






          active

          oldest

          votes








          1 Answer
          1






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          up vote
          0
          down vote



          accepted










          You're calling response.send(data) multiple times in forEach loop. That's not likely to do what you want, as you're only supposed to call send() once with then entire response. Instead, you should be collecting all the data to send in a single object, and call send() with that one object.






          share|improve this answer





















          • Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
            – SinishaS
            Nov 12 at 6:24

















          up vote
          0
          down vote



          accepted










          You're calling response.send(data) multiple times in forEach loop. That's not likely to do what you want, as you're only supposed to call send() once with then entire response. Instead, you should be collecting all the data to send in a single object, and call send() with that one object.






          share|improve this answer





















          • Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
            – SinishaS
            Nov 12 at 6:24















          up vote
          0
          down vote



          accepted







          up vote
          0
          down vote



          accepted






          You're calling response.send(data) multiple times in forEach loop. That's not likely to do what you want, as you're only supposed to call send() once with then entire response. Instead, you should be collecting all the data to send in a single object, and call send() with that one object.






          share|improve this answer












          You're calling response.send(data) multiple times in forEach loop. That's not likely to do what you want, as you're only supposed to call send() once with then entire response. Instead, you should be collecting all the data to send in a single object, and call send() with that one object.







          share|improve this answer












          share|improve this answer



          share|improve this answer










          answered Nov 12 at 6:17









          Doug Stevenson

          68.2k880100




          68.2k880100












          • Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
            – SinishaS
            Nov 12 at 6:24




















          • Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
            – SinishaS
            Nov 12 at 6:24


















          Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
          – SinishaS
          Nov 12 at 6:24






          Thanks Doug, I am sure that is the reason. If I don't see this, then it is time to finish for today :)
          – SinishaS
          Nov 12 at 6:24




















          draft saved

          draft discarded




















































          Thanks for contributing an answer to Stack Overflow!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.





          Some of your past answers have not been well-received, and you're in danger of being blocked from answering.


          Please pay close attention to the following guidance:


          • Please be sure to answer the question. Provide details and share your research!

          But avoid



          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function () {
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fstackoverflow.com%2fquestions%2f53256695%2fcloud-firestore-function-returns-only-one-document%23new-answer', 'question_page');
          }
          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          Bressuire

          Vorschmack

          Quarantine