-
-
Notifications
You must be signed in to change notification settings - Fork 40
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
feat(playground): add default query fn example #36
feat(playground): add default query fn example #36
Conversation
Run & review this pull request in StackBlitz Codeflow. |
...ages/playground/src/app/default-query-function-page/default-query-function-page.component.ts
Show resolved
Hide resolved
packages/playground/src/main.ts
Outdated
@@ -35,6 +38,25 @@ bootstrapApplication(AppComponent, { | |||
}); | |||
}, | |||
}, | |||
{ | |||
provide: QUERY_CLIENT_CONFIG, | |||
deps: [HttpClient], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You can remove the deps and use inject inside useFactory
c4cd16d
to
81fcff7
Compare
Can you please pull from master? |
81fcff7
to
8695af2
Compare
@hrvbernardic please fix the conflicts and pull master. |
8695af2
to
5e20144
Compare
@hrvbernardic, it only runs when you don't pass a queryFn right? the rest of the pages work fine. |
yeah I tried the others |
Merged. What's next :) |
haha, Ill take a look |
PR Checklist
Please check if your PR fulfills the following requirements:
PR Type
What kind of change does this PR introduce?
What is the current behavior?
Issue Number: #31
What is the new behavior?
Does this PR introduce a breaking change?
Other information