ref(ui): Remove qs imports, not a dependency

[scttcper]

We do not have qs as a direct dependency. It works anyway because we have it as a sub dependency, but we shouldn't import things that aren't dependencies. This could be a eslint rule we enable.

yarn why qs

=> Found "qs@6.11.0"
info Reasons this module exists
   - "webpack-dev-server#express" depends on it
   - Hoisted from "webpack-dev-server#express#qs"
   - Hoisted from "@rsdoctor#webpack-plugin#@rsdoctor#sdk#body-parser#qs"

[gggritso]

I was going to ask if qs and query-string have identical behaviour, but all the tests pass so I guess that's the answer I wanted

[scttcper]

i think array formatting is usually the only one that matters for us. EDIT: It doesn't seem to be the same

import queryString from 'query-string';

queryString.stringify({foo: [1, 2, 3]});
//=> 'foo=1&foo=2&foo=3'

qs.stringify({ a: ['b', 'c', 'd'] });
// 'a[0]=b&a[1]=c&a[2]=d'

[scttcper]

i don't see any arrays being formatted