Skip to content

Update query to include case sensitive flag

Administrator requested to merge insights/respect-case-sensitive into main

Created by: unclejustin

Closes https://github.com/sourcegraph/sourcegraph/issues/41120

Search removes case:yes from the query during passing. The backend needs it in the query. This PR adds it back if caseSensitive is true.

Test plan

Note: When I was testing locally, at some point my case-sensitive flag got "saved". So if you start a new search make sure that the case-sensitive button is not highlighted if.

Actual photo of the offender

Testing locally navigate here: https://sourcegraph.test:3443/search?q=context%3Aglobal+repo%3A%5Egithub%5C.com%2Fsourcegraph%2Fsourcegraph%24+file%3Aweb+file%3Acodeinsightsrouter++lang%3AtypeScript&patternType=standard&case=yes&groupBy=PATH&groupByUI=sidebar

Check the network tab in devtools and the case:yes should still be in the query variable.

It should not render a graph since there are no results for that search.

Navigate to https://sourcegraph.test:3443/search?q=context%3Aglobal+insights&patternType=standard&groupBy=REPO

Check the network tab in devtools and the case:yes should not be in the query variable.

It should render a graph since there are results for that search.

App preview:

Check out the client app preview documentation to learn more.

Merge request reports