rtk-incubator / rtk-query

Data fetching and caching addon for Redux Toolkit
https://redux-toolkit.js.org/rtk-query/overview
MIT License
626 stars 31 forks source link

Smarter handling optional parameters #146

Closed kahirokunn closed 3 years ago

kahirokunn commented 3 years ago
console.log(new URLSearchParams({ a: 'helloworld', b: undefined }).toString())
"a=helloworld&b=undefined"

This b=undefined should be unintentional, so undefined should be removed.

codesandbox-ci[bot] commented 3 years ago

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 8d6e1c9b060b475a90b82203d65d835e6d46f3fe:

Sandbox Source
React Configuration
React Typescript Configuration
rtk-query-demo Configuration
svelte-app-rtk-simplequery-demo Configuration
msutkowski commented 3 years ago

Thanks again @kahirokunn !!!!!!!

kahirokunn commented 3 years ago

Thank you too!!!