-
Notifications
You must be signed in to change notification settings - Fork 0
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
[Dataset page] : Filter by category in dataset page #91
Conversation
apps/datahub/src/app/dataset/dataset-information/dataset-information.component.html
Outdated
Show resolved
Hide resolved
apps/datahub/src/app/dataset/dataset-information/dataset-information.component.ts
Outdated
Show resolved
Hide resolved
apps/datahub/src/app/dataset/dataset-information/dataset-information.component.ts
Outdated
Show resolved
Hide resolved
a6cdb13
to
55fc703
Compare
88c0f4b
to
2ceb016
Compare
2ceb016
to
23e957a
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks @cmoinier for implementing this new filter!
I tested it locally and it works as expected.
Not sure if we should add a test here for the existence of this filter? WDYT?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for adding the tests! LGTM 👍
Description
This PR adds filtering when the user clicks on the category badges on the dataset page.
It also removes a duplicate style class.