-
-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Pass query args to prepareHeaders function #4638
Conversation
Review or Edit in CodeSandboxOpen the branch in Web Editor • VS Code • Insiders |
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 3a22c67:
|
✅ Deploy Preview for redux-starter-kit-docs ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
I think I'd probably prefer adding |
f3b486a
to
9d43ea0
Compare
Thanks for the feedback Mark. I've modified the code to pass into the second args object. |
Looks like we have uses of both Given that, I'm going to switch this back to |
Closes #4238
Passes args in the query function to the
prepareHeaders
call as the third argument.In the base fetch function, I renamed the first argument from
arg
toargs
to make it consistent with the name in the types and docs.