search

ScottyLabs / cmu_lost_and_found_v2

CMU Lost and Found website
2 stars 3 forks source link

added location based searching for issue #85 #92

Closed ndnrlg closed 2 years ago

elouiestudent commented 2 years ago

Looks like the location-based searching is working, but we don't want to override description-based searching with it. Could you try implementing a drop-down button that toggles between location-based and description-based searching in the search bar?

elouiestudent commented 2 years ago

On second thought, we should instead just have the search bar search through both the description and the location simultaneously without the toggle.

ndnrlg commented 2 years ago

On second thought, we should instead just have the search bar search through both the description and the location simultaneously without the toggle.

Got you- will make that change! Just made a new commit

elouiestudent commented 2 years ago

LGTM!