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

[RED-32] Remove array form for configureStore.middleware argument #3748

Closed
markerikson opened this issue Sep 25, 2023 · 0 comments
Closed

[RED-32] Remove array form for configureStore.middleware argument #3748

markerikson opened this issue Sep 25, 2023 · 0 comments
Assignees
Milestone

Comments

@markerikson
Copy link
Collaborator

markerikson commented Sep 25, 2023

We've seen too many cases where users call configureStore({reducer, middleware: [myMiddleware]}), and accidentally blow away the defaults.

Let's remove that form and only allow callbacks. We already did that for enhancers, so same idea.

Users can still do middleware: () => [myMiddleware] if they really want to, but that at least forces them to make use of the callback form, and reduces the chance of it being a mistake.

RED-32

@markerikson markerikson changed the title Remove array form for configureStore.middleware argument [RED-32] Remove array form for configureStore.middleware argument Sep 25, 2023
@markerikson markerikson added this to the 2.0 milestone Sep 25, 2023
@markerikson markerikson closed this as not planned Won't fix, can't repro, duplicate, stale Mar 26, 2024
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

No branches or pull requests

2 participants