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

[Asset Selection] Make submitting an invalid query with only a single value convert to a key substring filter. #28435

Merged
merged 2 commits into from
Mar 12, 2025

Conversation

salazarm
Copy link
Contributor

Summary & Motivation

A lot of users are using their muscle memory and doing:

  1. Type partial asset key
  2. Hit enter
  3. Error -> No results
  4. Give up.

Lets just do the right thing for these users by automatically converting their invalid query into a valid one!

How I Tested these changes

Screen.Recording.2025-03-12.at.1.51.11.PM.mov

Copy link

Deploy preview for dagit-core-storybook ready!

✅ Preview
https://dagit-core-storybook-1s53m9gm3-elementl.vercel.app
https://salazarm-make-old-flow-good.core-storybook.dagster-docs.io

Built with commit cf6252c.
This pull request is being automatically deployed with vercel-action

@salazarm salazarm merged commit 30c1e2b into master Mar 12, 2025
6 of 7 checks passed
@salazarm salazarm deleted the salazarm/make-old-flow-good branch March 12, 2025 17:54
salazarm added a commit that referenced this pull request Mar 12, 2025
… value convert to a key substring filter. (#28435)

## Summary & Motivation

A lot of users are using their muscle memory and doing:
1. Type partial asset key
2. Hit enter
3. Error -> No results
4. Give up.

Lets just do the right thing for these users by automatically converting
their invalid query into a valid one!

## How I Tested these changes


https://github.com/user-attachments/assets/f8533af6-81db-4b38-8e2e-ed2a6e18640e
salazarm added a commit that referenced this pull request Mar 12, 2025
…lue convert to a name substring filter. (#28437)

## Summary & Motivation
Similar to #28435


## How I Tested These Changes
locally
salazarm added a commit that referenced this pull request Mar 12, 2025
…lue convert to a name substring filter. (#28437)

## Summary & Motivation
Similar to #28435


## How I Tested These Changes
locally
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.

2 participants