Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/select dropdown map filters #40

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

Conversation

MilesHeise
Copy link
Contributor

adds basic version of dropdown filters for the map

notes:

  • the multi-filter might want different options/more options/correctly spelled options

  • the filter for things like wic/snap might be better as individual checkboxes? also I had to test functionality using 'saturday' as the filter rather than things like wic or snap because there is currently no data with 'true' for any of them

  • all of it could use a refactor probably, very quickly done

@MilesHeise
Copy link
Contributor Author

since this one hadn't gone through from 2 weeks ago, had to add onto this branch rather than a new PR. second push includes removing the dummy data from the details page and updating the select tag for towns to be a typeahead select as requested

@saylorsd
Copy link
Contributor

saylorsd commented Mar 4, 2020

@MilesHeise it looks like there's a merge conflict. could update your fork to so we can merge it in clean?

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

Successfully merging this pull request may close these issues.

2 participants