ScottyLabs / cmu_lost_and_found_v2

CMU Lost and Found website

2 stars 3 forks source link

Fix search dropdown #184

Closed elouiestudent closed 2 years ago

elouiestudent commented 2 years ago

Description

This pull request fixes a bug where the search bar only searches by keyword even when other search settings are toggled.

Type of change

[x] Bug fix (non-breaking change which fixes an issue)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to not work as expected)
[ ] This change requires a documentation update

How Has This Been Tested?

[x] Check search bar settings work as expected

Test Configuration:

Node.js version: 14.15.4
Desktop/Mobile: Desktop
Browser: Google Chrome

Checklist:

[x] My code follows the style guidelines of this project
[x] I have performed a self-review of my code
[x] I have commented my code, particularly in hard-to-understand areas
[ ] I have made corresponding changes to the documentation
[x] My changes generate no new warnings
[ ] I have added tests that prove my fix is effective or that my feature works
[ ] New and existing unit tests pass locally with my changes