Skip to content

Search Upgrades - #231

Draft
vkoves wants to merge 13 commits into
mainfrom
search-upgrades
Draft

Search Upgrades#231
vkoves wants to merge 13 commits into
mainfrom
search-upgrades

Conversation

@vkoves

@vkoves vkoves commented Aug 29, 2025

Copy link
Copy Markdown
Owner

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

image

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.

Home Header
image image

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:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • 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
  • If I added a large new feature, I added it to the release notes (ReleaseNotes.vue)

Data Update (if applicable):

@vkoves vkoves added enhancement New feature or request frontend Frontend/UI work labels Aug 29, 2025
@netlify

netlify Bot commented Aug 29, 2025

Copy link
Copy Markdown

Deploy Preview for radiant-cucurucho-d09bae ready!

Name Link
🔨 Latest commit e3e0614
🔍 Latest deploy log https://app.netlify.com/projects/radiant-cucurucho-d09bae/deploys/6a72b9b21ef92400088ae34f
😎 Deploy Preview https://deploy-preview-231--radiant-cucurucho-d09bae.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@vkoves

vkoves commented Aug 29, 2025

Copy link
Copy Markdown
Owner Author

I'm seeing the out of memory issue again, so I think we need to add is_new as a Python property, so we don't need to fetch historic data to determine that and do filtering.

hunter-yeago

This comment was marked as outdated.

@vkoves

vkoves commented Oct 13, 2025

Copy link
Copy Markdown
Owner Author

@hunter-yeago - this is an old draft PR with some new search stuff, so I actually need to merge with master and take advantage of the new fields I've added in. I think maybe you got something confused, I wanted review on PR #245! (I just moved that to open)

@hunter-yeago

Copy link
Copy Markdown
Collaborator

@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.

Comment thread src/pages/Search.vue Fixed
Comment thread src/pages/Search.vue Fixed
@vkoves

vkoves commented Feb 4, 2026

Copy link
Copy Markdown
Owner Author

Talking in person, the expand collapse is a bit overkill, I should move the search stuff back to one line

@github-actions

github-actions Bot commented Jul 22, 2026

Copy link
Copy Markdown

🎭 Playwright Visual Test Report

Run on commit 0d04c51 at Aug 4, 2026, 11:23 PM CDT

❌ 1/22 tests failed

We've detected visual changes on: All Electric Page

Failed Tests (1)
  • All Electric Page (Mobile Chrome)
    6079 pixels (ratio 0.01 of all image pixels) are different
Passed Tests (21)
  • All Electric Building (Print Page 1)
  • Mix Building (Merch Mart) (Print Page 1)
  • No Image Building (Print Page 1)
  • All Electric Building
  • Mix Building (Merch Mart)
  • No Image Building
  • Building Owner (IIT)
  • Search
  • Home
  • All Electric Page
  • Ward 47

📦 Download Full Report | 🔍 View Run Details

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request frontend Frontend/UI work

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants