Skip to content
This repository has been archived by the owner on Jul 23, 2019. It is now read-only.

Search #9

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Search #9

wants to merge 8 commits into from

Conversation

rdev0rigin
Copy link
Collaborator

This PR introduces a search feature and refactored the filter UI.
screen shot 2018-10-11 at 8 39 46 pm

screen shot 2018-10-11 at 8 39 12 pm

@miklb
Copy link
Contributor

miklb commented Oct 12, 2018

Great start! One question about the filtering
search_example

Here Walton comes in 3rd in the results. Just curious how you are sorting that.

@rdev0rigin
Copy link
Collaborator Author

Good question. :) I have it searching through everything in the dataset, I need to tweak the search criteria and fields a little, Freeport is Walton county, I have the counties ranked higher I guess then the name..

@rdev0rigin
Copy link
Collaborator Author

rdev0rigin commented Oct 12, 2018

@miklb I fixed the filters and I made the search criteria a little stricter, though Im not sure why Im getting higher ranks on some of the data then others that don't seem like they should.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants