• Robert Knight's avatar
    Omit query params with `undefined` values from API calls · 71e9385e
    Robert Knight authored
    The previous version stated that it aimed to match the behavior of the
    `query-string` package but it was actually tested against Node's
    `querystring` package, which has slightly different behavior for this
    edge case. It omits parameters with `undefined` values and renders empty
    parameters for `null` values.
    
    The notebook view was calling `api.search({ uri: undefined, ... })` and
    this translated to `/api/search?uri=&...` which caused an error.
    
    Omit parameters instead in this case.
    71e9385e
Name
Last commit
Last update
..
test Loading commit data...
collections.js Loading commit data...
copy-to-clipboard.js Loading commit data...
disable-opener-for-external-links.js Loading commit data...
dom.js Loading commit data...
immutable.js Loading commit data...
memoize.js Loading commit data...
oauth-client.js Loading commit data...
observe-element-size.js Loading commit data...
pagination.js Loading commit data...
postmessage-json-rpc.js Loading commit data...
random.js Loading commit data...
retry.js Loading commit data...
search-filter.js Loading commit data...
sentry.js Loading commit data...
time.js Loading commit data...
typing.js Loading commit data...
unicode.js Loading commit data...
url.js Loading commit data...
watch.js Loading commit data...