search

maxlemieux / quokka

Project 3 "Quokka"
MIT License
0 stars 2 forks source link

Main feed auto-loading #35

Closed maxlemieux closed 4 years ago

maxlemieux commented 4 years ago

The main feed needs to load automatically, ideally.

Right now it uses a button click to trigger loading.

Earlier attempts to auto-load triggered endless loops. This can probably be addressed by managing the contents of the list with state at a higher level.

maxlemieux commented 4 years ago

Stretch goal.

maxlemieux commented 4 years ago

If we add a [searchResults] parameter to the useEffect, it stops the endless loop (by only running the effect on initial render or when searchResults is updated).

However, the Get Suggestions button is kinda fun to click and I'm thinking we can leave it in the app for now.