-
Notifications
You must be signed in to change notification settings - Fork 4
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
[DT-776] Filter by Data Type #2726
Conversation
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.
How many data types do we expect? I know the long list is on dev, are we expecting much fewer on prod? If it is going to be long, I'd love to think about limiting the amount shown, and figuring out a see more
story.
We have 8 by default, but a user can add their own if they really want (although not all 8 may be shown since it's dynamically populated from the datasets). I agree that long term if the list gets long we should add a separate story to "see more". |
Great. I'll flag that as a potential future ticket for Victoria |
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.
LGTM
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.
this looks good to me, although is there any kind of testing we want to do here?
Good idea, let me work on increasing our unit test coverage. |
4c45f09
to
48a21a9
Compare
48a21a9
to
3c8db98
Compare
Addresses
https://broadworkbench.atlassian.net/browse/DT-776
Summary
Adds a filter for the Data Type (note that in dev we have a bunch of weird selections)
Screen.Recording.2024-11-14.at.1.19.00.PM.mov
Have you read Terra's Contributing Guide lately? If not, do that first.