Search Upgrades - #231
Conversation
The new filter is quite slow though, so we might want to add that as a BE python step
✅ Deploy Preview for radiant-cucurucho-d09bae ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
|
I'm seeing the out of memory issue again, so I think we need to add |
|
@hunter-yeago - this is an old draft PR with some new search stuff, so I actually need to merge with |
|
@vkoves Ah. My bad. You know, they don't have schools in Florida. So I'm still learning how to read. Taking a look at the other PR now. |
|
Talking in person, the expand collapse is a bit overkill, I should move the search stuff back to one line |
🎭 Playwright Visual Test ReportRun on commit ❌ 1/22 tests failed We've detected visual changes on: All Electric Page Failed Tests (1)
Passed Tests (21)
|
Resolves #18
Improves build performance, since the CSV won't change unless building data changes
Description
Made two big search improvements.
1. More Search Filters
Added a "New" filter and "All Electric" filter. I think in the future I might want to rework this to "Started Reporting In" or something like that, and add other filters, but this is a good start
2. Search Autocomplete!
Tackled issue #18 with autocomplete on the search page and in the header on other pages, making it a lot easier to find buildings and see issues with addresses, making travelling between buildings more fluid.
Testing Instructions
Please describe the tests/QA that you did to verify your changes. Provide instructions so we can reproduce.
Please also list any relevant details for your test configuration
Checklist:
Data Update (if applicable):