Skip to content

Conversation

alexmotoc
Copy link
Contributor

I've opted for updating the types of selectCachedArgsForQuery to support infinite queries rather than creating a new selectCachedArgsForInfiniteQuery method. Considering this is a bug fix for existing functionality, I don't think any doc updates are required here

Fixes #4875

Copy link

codesandbox bot commented Mar 7, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@alexmotoc alexmotoc changed the title add initite query type support for selectCachedArgsForQuery add infinite query type support for selectCachedArgsForQuery Mar 7, 2025
Copy link

codesandbox-ci bot commented Mar 7, 2025

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 00553f2:

Sandbox Source
@examples-query-react/basic Configuration
@examples-query-react/advanced Configuration
@examples-action-listener/counter Configuration
rtk-esm-cra Configuration

@alexmotoc alexmotoc force-pushed the cached-args-for-query-types branch from ca29eae to 00553f2 Compare March 7, 2025 18:42
Copy link

netlify bot commented Mar 7, 2025

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

Name Link
🔨 Latest commit ca29eae
🔍 Latest deploy log https://app.netlify.com/sites/redux-starter-kit-docs/deploys/67cb3d6d3d580d0008826c5b
😎 Deploy Preview https://deploy-preview-4880--redux-starter-kit-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

Copy link

netlify bot commented Mar 7, 2025

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

Name Link
🔨 Latest commit 00553f2
🔍 Latest deploy log https://app.netlify.com/sites/redux-starter-kit-docs/deploys/67cb3e2a8b09850008977c1d
😎 Deploy Preview https://deploy-preview-4880--redux-starter-kit-docs.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@markerikson
Copy link
Collaborator

Thanks, beat me to it!

@markerikson markerikson merged commit a59f90c into reduxjs:master Mar 7, 2025
47 checks passed
@alexmotoc
Copy link
Contributor Author

You beat me to the matchers type improvements earlier this week so it was only fair I'd do the same!

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.

More infinite query missing types
2 participants