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

Update part-5-async-logic.md #4230

Merged
merged 1 commit into from
Dec 7, 2021
Merged

Conversation

VanoAkofashvili
Copy link
Contributor

Maybe it should be a addDefaultCase() not addDefault()


name: 📝 Documentation Fix
about: Fixing a problem in an existing docs page

Checklist

  • [ x ] Is there an existing issue for this PR?
    • link issue here
  • [ x ] Have the files been linted and formatted?

What docs page needs to be fixed?

  • Section:
  • Async Logic and Data Fetching
  • Page:
  • Detailed Explanation: Adding Extra Reducers to Slices

What is the problem?

Incorrect function call (builder.addDefault())

What changes does this PR make to fix the problem?

Maybe it should be a addDefaultCase() not addDefault()
@codesandbox-ci
Copy link

codesandbox-ci bot commented Dec 7, 2021

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 fd9e783:

Sandbox Source
Vanilla Configuration
Vanilla Typescript Configuration

@netlify
Copy link

netlify bot commented Dec 7, 2021

✔️ Deploy Preview for redux-docs ready!

🔨 Explore the source changes: fd9e783

🔍 Inspect the deploy log: https://app.netlify.com/sites/redux-docs/deploys/61af23e93f37150008d958db

😎 Browse the preview: https://deploy-preview-4230--redux-docs.netlify.app

@markerikson
Copy link
Contributor

Yep, thanks!

@markerikson markerikson merged commit f81817a into reduxjs:master Dec 7, 2021
nevilm-lt pushed a commit to nevilm-lt/redux that referenced this pull request Apr 22, 2022
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