remove SearchOptions, clean up search URL generation code, add tests (!1452) · Merge requests · Warren Gifford / sourcegraph

Warren Gifford requested to merge rm-searchoptions into master Dec 15, 2018

Created by: sqs

Removes SearchOptions, which used to have both q and sq properties because we used to have a separate "scope query" (sq) URL param. This was removed in around Jan-Feb 2018 and is no longer necessary to keep around for backcompat. Now the SearchOptions can just be a single string, the query itself.

Also dedupes buildSearchURLQuery between the web app and browser extension.

Also adds tests! 🎉

remove SearchOptions, clean up search URL generation code, add tests

Merge request reports