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.

Sign up for GitHub

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

Jump to bottom

Allow filtering to type of vaccine #4

Open
livgust opened this issue Feb 12, 2021 · 1 comment · Fixed by #13
Open

Allow filtering to type of vaccine #4

livgust opened this issue Feb 12, 2021 · 1 comment · Fixed by #13
Labels
UX enhancement

Comments

@livgust
Copy link
Owner

livgust commented Feb 12, 2021

Some people are looking to book their second appointment and have to find one for the same vaccine manufacturer as their first appointment.

This would likely mean an update to the back-end scrapers to ensure that they pass that information if available.

If we add enough filters and sorters, we likely will want some kind of gear or menu for filters so they don't clutter the page.
@livgust livgust added the enhancement New feature or request label Feb 12, 2021
@linusmarco
Copy link
Collaborator

I'm working on this one. Should have a PR in this weekend. As part of this I'm doing a little refactor to group all filters into a component t separate from the main site list.

@linusmarco linusmarco self-assigned this Feb 13, 2021
linusmarco added a commit to linusmarco/macovidvaccines.com that referenced this issue Feb 13, 2021
@linusmarco
Add vaccine type filter (livgust#4), move all filters to panel
07eebe6
@linusmarco linusmarco mentioned this issue Feb 13, 2021
Merged
@harcod harcod linked a pull request Feb 18, 2021 that will close this issue
Add vaccine type filter, move all filters to separate component #13
Merged
livgust added a commit that referenced this issue Feb 19, 2021
@linusmarco @livgust
@indigo-mcardarelli @harcod @johnhawkinson
Add vaccine type filter, move all filters to separate component (#13)
fda595e 
* remove .eslintcache from tracking

* Add vaccine type filter (#4), move all filters to panel

* address code review comments

* more code review revisions

* remove console.log

* WIP updating tests - one still failing

* Add warning when sites are restricted

* moving to new line

* fixed test

* added missing comma, womp womp

* get filters into responsive drawer

* Put filters into a responsive drawer (#42)

* Update docs to make PRs to develop

* update develop again (#32)

* Improve HTML semantics of a couple key elements

* Tweaks to elements

* Add eligibility accordion

* Remove extraneous prettierrc and prettify README.md

* Link to mass.gov/covidvaccine gave a security warning becuase the state didn't redirect properly. Link changed to a non-redirecting link.

* Update eligibility for feb 18, small refactor

* link to senior housing details

Co-authored-by: Matthew Cardarelli <mattrc7@gmx.com>
Co-authored-by: Jake Rogers <jake@dataplusmath.com>
Co-authored-by: Rob Harris <rob@harcod.com>
Co-authored-by: John Hawkinson <jhawk@alum.mit.edu>

* get filters into responsive drawer

Co-authored-by: Olivia Adams <liv.gust@gmail.com>
Co-authored-by: Matthew Cardarelli <mattrc7@gmx.com>
Co-authored-by: Jake Rogers <jake@dataplusmath.com>
Co-authored-by: Rob Harris <rob@harcod.com>
Co-authored-by: John Hawkinson <jhawk@alum.mit.edu>

* get new filter panel checkbox design working

Co-authored-by: Olivia Adams <liv.gust@gmail.com>
Co-authored-by: Matthew Cardarelli <mattrc7@gmx.com>
Co-authored-by: Jake Rogers <jake@dataplusmath.com>
Co-authored-by: Rob Harris <rob@harcod.com>
Co-authored-by: John Hawkinson <jhawk@alum.mit.edu>
@livgust livgust added UX enhancement and removed enhancement New feature or request labels Mar 1, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
UX enhancement
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add vaccine type filter, move all filters to separate component linusmarco/macovidvaccines.com
2 participants
@livgust @linusmarco