-
Notifications
You must be signed in to change notification settings - Fork 199
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
feat(datasets): multi-select on span / traces tables #3236
Conversation
bottom selection toolbar create a span selection toolbar Add datasets query get selection working refactor: use input and output types for create datasets mutation (#3242) WIP final refactor, typecheck
4988f4f
to
4f927f5
Compare
onDatasetCreated={(newDataset) => { | ||
notifySuccess({ | ||
title: "Dataset created", | ||
message: `${payload.dataset.name} has been successfully created.`, | ||
message: `${newDataset.name} has been successfully created.`, | ||
action: { | ||
text: "Go to Dataset", | ||
onClick: () => { | ||
navigate(`/datasets/${payload.dataset.id}`); | ||
navigate(`/datasets/${newDataset.id}`); |
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.
Nice 👍
const datasets = useMemo(() => { | ||
let datasets = data.datasets.edges.map((edge) => edge.dataset); | ||
if (search) { | ||
datasets = datasets.filter((dataset) => | ||
dataset.name.toLowerCase().includes(search.toLowerCase()) | ||
); | ||
} | ||
return datasets; |
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.
Assuming this is the part we would like to make a LIKE query for. Do we have a ticket for that yet?
const [data] = useRefetchableFragment( | ||
graphql` | ||
fragment DatasetSelectorPopoverContent_datasets on Query | ||
@refetchable(queryName: "DatasetSelectorPopoverContentDatasetsQuery") { | ||
datasets { | ||
edges { | ||
dataset: node { | ||
id | ||
name | ||
} | ||
} | ||
} | ||
} | ||
`, | ||
props.query | ||
); |
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.
Is this a place where we should add debounce?
alignItems="center" | ||
flex="1 1 auto" | ||
> | ||
<Text>No datasets found</Text> |
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.
Could use some padding here.
resolves #3202
Adds the ability to add spans as examples to a dataset from the tables