a little confused about setproject hooks code about todoist HOT 7 CLOSED

const allProjects = snapshot.docs.map((project) => ({
          name: project.data().name,
          userId: project.data().userId,
          projectId: project.data().projectId,
          docId: project.id,
        }));

This solves the RErendering as Stringify conditions work only when sequence is maintained, firestore is Sending data in random order at times, So avoid spread operaetor here would save 50k firebase calls for you.

from todoist.

Hey @apawn, so what happens here is that when useProjects is called, it goes ahead and makes a request to Firebase to get the data. If projects changes, it will re-run the useEffect so that's why I've added a JSON.stringify in there, so make sure it doesn't go into an infinite loop. If you call setProjects it will go ahead call Firebase again. Hopefully, that makes sense, if not let me know and I'll try my best to help :)

from todoist.

I also got confused here. Is it necessary to use .get and .onSnapshot also?
Handling Firebase like a Rest API, and maintaining a local state ignoring live changes would clarify it a lot.

from todoist.

@BlackSonic For this particular hook it does ignore live changes and just gets you a snapshot of what you have in the database.

from todoist.

@karlhadwen i mean here, it is not necessary to subscribe to the changes of firebase, we just use it for storage. and when calling useProject, you can save the data to firebase, and after that, update the state, which will cause UI update. when first time mounting the App Component, reading data from firebase.

from todoist.

@apawn you will see that it's not actually updating to firebase or subscribing listening to events, it takes a snapshot, unlike the tasks that are subscribing and listening for changes in tasks.

from todoist.

Something is causing it to render projects again and again is there a possibility of circular dependency here

from todoist.