Skip to content
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

fix: Add missing fetcherOptions to queryFn #279

Closed
wants to merge 2 commits into from

Conversation

rajzik
Copy link
Contributor

@rajzik rajzik commented Feb 3, 2025

Fixes: #278

@fabien0102 fabien0102 force-pushed the add-missing-fetcher-options branch from 18fd9c5 to fc951c4 Compare February 3, 2025 15:09
@rajzik rajzik force-pushed the add-missing-fetcher-options branch from 7cf9a47 to b7b80d8 Compare February 4, 2025 07:30
@rajzik
Copy link
Contributor Author

rajzik commented Feb 4, 2025

@fabien0102 I think this PR is ready.

fabien0102 added a commit that referenced this pull request Feb 4, 2025
* fix!: deep merge variables with fetcherOptions from context.

Closed #226 #227 #279 #278

* Add @dan-cooke as contributor
@fabien0102 fabien0102 closed this Feb 4, 2025
@fabien0102
Copy link
Owner

Sorry I took this opportunity to rework an old PR to add deepMerge, anyway, no need to pull fetcherOptionTypes, the types are already merge in variables.

Please tell me if you find anything fishy in #284

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Context fetcherOptions is ignored for queryFn calls
2 participants