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

Fix redux-thunk imports in docs examples #4779

Merged
merged 1 commit into from
Mar 4, 2025

Conversation

sinyakov
Copy link
Contributor

@sinyakov sinyakov commented Mar 1, 2025


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

Checklist

  • Is there an existing issue for this PR?
    • no, but seems like trivial changes
  • Have the files been linted and formatted?

What docs page needs to be fixed?

What is the problem?

Thunk middleware is imported as default in docs. Meanwhile [email protected] provides named exports.

What changes does this PR make to fix the problem?

Imports are fixed in docs and examples.

Copy link

codesandbox-ci bot commented Mar 1, 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.

Copy link

netlify bot commented Mar 1, 2025

Deploy Preview for redux-docs ready!

Name Link
🔨 Latest commit 9b1e293
🔍 Latest deploy log https://app.netlify.com/sites/redux-docs/deploys/67c44170c1d90d00080f2265
😎 Deploy Preview https://deploy-preview-4779--redux-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.

@sinyakov sinyakov force-pushed the redux-thunk-3-examples branch from 6a6c2dd to 7f8ce4a Compare March 1, 2025 22:02
@sinyakov sinyakov force-pushed the redux-thunk-3-examples branch from 7f8ce4a to 9b1e293 Compare March 2, 2025 11:30
@timdorr
Copy link
Member

timdorr commented Mar 4, 2025

Thanks!

@timdorr timdorr merged commit 9f3de74 into reduxjs:master Mar 4, 2025
12 checks passed
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