Firebase init #9
Conversation
| import firebase from 'firebase/app' | ||
| import 'firebase/firestore' | ||
|
|
||
| if (!firebase.apps.length) { |
There was a problem hiding this comment.
What is this conditional checking? To see if there's a firebase app?
There was a problem hiding this comment.
Yeah, this file will be imported many times and we only want a single app
There was a problem hiding this comment.
Maybe a good place to use a singleton?
There was a problem hiding this comment.
this is the singleton
There was a problem hiding this comment.
oh lmao all the singletons i've seen before had a slightly different structure but this works
src/App.js
Outdated
|
|
||
| // example getting all hackathons data | ||
| useEffect(() => { | ||
| db.collection('Hackathons').get().then(querySnapshot => { |
There was a problem hiding this comment.
Would it make sense to make a collection for each hackathon so we don't grab more info than we need?
There was a problem hiding this comment.
This is just an example call, we can defs just grab one hackathon's data.
src/App.js
Outdated
| // example getting all hackathons data | ||
| useEffect(() => { | ||
| db.collection('Hackathons').get().then(querySnapshot => { | ||
| const paths = querySnapshot.docs.map(doc => { |
There was a problem hiding this comment.
Why is this var called paths
Closes #7