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

Hierarchical filters #233

Merged
merged 24 commits into from
Jan 19, 2022
Merged

Hierarchical filters #233

merged 24 commits into from
Jan 19, 2022

Conversation

ptbrowne
Copy link
Collaborator

@ptbrowne ptbrowne commented Nov 26, 2021

DimensionValuesMultiFilter is now displayed as a tree
The hierarchy is for now hardcoded for the bathing site dataset

Various refactors were necessary on widgets like the selectbox to be able to cleanly insert this feature.

See the result here : https://visualize-ad-feat-hiera-6sn1sd.herokuapp.com/en/create/zU4PAOQ38gfL

@ptbrowne ptbrowne changed the base branch from main to refactor/filter-components November 26, 2021 16:57
@jstcki jstcki temporarily deployed to visualize-ad-feat-hiera-6sn1sd November 26, 2021 16:58 Inactive
Base automatically changed from refactor/filter-components to feat/hierarchical-filters-main November 30, 2021 09:21
@ptbrowne ptbrowne force-pushed the feat/hierarchical-filters branch from d1e1288 to 79ab43d Compare January 18, 2022 09:17
@ptbrowne ptbrowne had a problem deploying to visualize-ad-feat-hiera-hivdhg January 18, 2022 09:21 Failure
@ptbrowne ptbrowne changed the base branch from feat/hierarchical-filters-main to main January 18, 2022 09:22
@ptbrowne ptbrowne force-pushed the feat/hierarchical-filters branch from 79ab43d to 7e75c75 Compare January 18, 2022 09:22
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-hivdhg January 18, 2022 09:22 Inactive
@ptbrowne ptbrowne force-pushed the feat/hierarchical-filters branch from 7e75c75 to 7dd70e9 Compare January 18, 2022 11:30
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-u9w9ra January 18, 2022 11:30 Inactive
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-dl6fvm January 18, 2022 13:59 Inactive
@ptbrowne ptbrowne marked this pull request as ready for review January 18, 2022 14:02
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-ktnzg5 January 18, 2022 14:05 Inactive
@ptbrowne ptbrowne force-pushed the feat/hierarchical-filters branch from fca9f2d to 9387a5a Compare January 18, 2022 14:11
@ptbrowne ptbrowne changed the title Hierarchical filters draft Hierarchical filters Jan 18, 2022
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-wo65au January 18, 2022 14:20 Inactive
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-wo65au January 18, 2022 14:35 Inactive
@ptbrowne ptbrowne requested a review from bprusinowski January 18, 2022 14:46
Copy link
Collaborator

@bprusinowski bprusinowski left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! PR is very clear and I like that you improved some of the code related to charts along the way :) 👍

@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-wo65au January 19, 2022 09:06 Inactive
@ptbrowne ptbrowne force-pushed the feat/hierarchical-filters branch from 674e06f to e1c7a03 Compare January 19, 2022 09:06
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-wo65au January 19, 2022 09:06 Inactive
@ptbrowne ptbrowne force-pushed the feat/hierarchical-filters branch from e1c7a03 to 7bfb59a Compare January 19, 2022 10:18
@ptbrowne ptbrowne temporarily deployed to visualize-ad-feat-hiera-wo65au January 19, 2022 10:18 Inactive
@ptbrowne ptbrowne merged commit a2b02ad into main Jan 19, 2022
@ptbrowne ptbrowne deleted the feat/hierarchical-filters branch January 19, 2022 12:16
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.

3 participants