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

docs: Clarify documentation about the overrideExisting flag in injectEndpoints #2696

Merged
merged 1 commit into from
Sep 14, 2022

Conversation

PauloRSF
Copy link
Contributor

Proposed changes

Clarify in the injectEndpoints docs that the overrideExisting flag actually controls whether endpoints are overridden or not in development.

Closes #2694

@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 2abbe74:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration
rsk-github-issues-example Configuration
@examples-query-react/basic Configuration
@examples-query-react/advanced Configuration
@examples-action-listener/counter Configuration

@netlify
Copy link

netlify bot commented Sep 14, 2022

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

Name Link
🔨 Latest commit 2abbe74
🔍 Latest deploy log https://app.netlify.com/sites/redux-starter-kit-docs/deploys/6321d794bc5525000872a9ad
😎 Deploy Preview https://deploy-preview-2696--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 settings.

@markerikson
Copy link
Collaborator

Thanks!

@markerikson markerikson merged commit 7c3129c into reduxjs:master Sep 14, 2022
@PauloRSF PauloRSF deleted the clarify-overrideexisting-docs branch September 14, 2022 14:53
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.

RTK Query: Possibly misleading documentation for overrideExisting option in endpoint injection
2 participants