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

bug-1909637: allow searching for files from regular and try symbol uploads #2992

Merged
merged 1 commit into from
Aug 19, 2024

Conversation

willkg
Copy link
Contributor

@willkg willkg commented Aug 14, 2024

Previously, you could sort of search for try uploads by searching for "try/" in the key. But you couldn't search for regular uploads.

We recently changed the key data in the model, so "try/" no longer appears in the key which breaks this flimsy filtering method.

This introduces a new upload_type filter field that lets you filter on nothing (""), try upload files ("try"), and regular upload files ("regular").

…loads

Previously, you could sort of search for try uploads by searching for
"try/" in the key. But you couldn't search for regular uploads.

We recently changed the key data in the model, so "try/" no longer
appears in the key which breaks this flimsy filtering method.

This introduces a new upload_type filter field that lets you filter on
nothing (""), try upload files ("try"), and regular upload files
("regular").
@willkg willkg force-pushed the willkg-bug-1909637-search-try branch from f8b4b52 to 7138777 Compare August 19, 2024 16:23
@willkg willkg marked this pull request as ready for review August 19, 2024 16:28
@willkg willkg requested a review from a team as a code owner August 19, 2024 16:28
@willkg willkg requested a review from relud August 19, 2024 16:28
@willkg willkg merged commit d7e6d39 into main Aug 19, 2024
2 checks passed
@willkg willkg deleted the willkg-bug-1909637-search-try branch August 19, 2024 16:43
@willkg
Copy link
Contributor Author

willkg commented Aug 19, 2024

Thank you!

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