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

Search#9

Open
rdev0rigin wants to merge 8 commits intohurricane-response:masterfrom
rdev0rigin:search
Open

Search#9
rdev0rigin wants to merge 8 commits intohurricane-response:masterfrom
rdev0rigin:search

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