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

Expose useLazyQuerySubscription for individual endpoints #1017

Conversation

Shrugsy
Copy link
Collaborator

@Shrugsy Shrugsy commented Apr 25, 2021

The typing for QueryHooks claimed to include useLazyQuerySubscription, which appeared to expose the hook for individual endpoints. However in the react module, it was never used, so something like api.endpoints.getPosts.useLazyQuerySubscription would show the corresponding type, but would actually be undefined.

This PR makes the react modules attach useLazyQuerySubscription to query endpoints, in line with the types.

@codesandbox-ci
Copy link

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 0277cc6:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration
rsk-github-issues-example Configuration

@netlify
Copy link

netlify bot commented Apr 25, 2021

Deploy preview for redux-starter-kit-docs ready!

Built with commit 0277cc6

https://deploy-preview-1017--redux-starter-kit-docs.netlify.app

Copy link
Member

@phryneas phryneas left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

good catch!

@markerikson markerikson merged commit 9beabae into reduxjs:feature/v1.6-integration Apr 25, 2021
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.

3 participants