Skip to content

Conversation

mattchannn
Copy link
Contributor

As we can put keepUnusedDataFor in both createApi and endpoints level hence we need to correct examples for each of the use case.

Due to the constraint from remark-typescript-tools, I am not able to have nested identifier in a Typescript type hence I put the example on the outermost level which I think no harm

Copy link

codesandbox bot commented Aug 29, 2025

Review or Edit in CodeSandbox

Open the branch in Web EditorVS CodeInsiders

Open Preview

@mattchannn mattchannn changed the title fix: incorrect position of keepUnusedDataFor for two scenarios fixdoc: incorrect position of keepUnusedDataFor for two scenarios Aug 29, 2025
Copy link

codesandbox-ci bot commented Aug 29, 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 469a581:

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

Copy link

netlify bot commented Aug 29, 2025

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

Name Link
🔨 Latest commit 469a581
🔍 Latest deploy log https://app.netlify.com/projects/redux-starter-kit-docs/deploys/68b6550d82d1a300081e6ade
😎 Deploy Preview https://deploy-preview-5062--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 project configuration.

@markerikson markerikson merged commit ab346b9 into reduxjs:master Sep 2, 2025
63 checks passed
@markerikson
Copy link
Collaborator

Thanks!

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.

2 participants