Upgrade to Rect Redux 8 & Redux Toolkit

[dependabot]

Bumps react-redux from 7.2.8 to 8.0.5.

Release notes

Sourced from react-redux's releases.

v8.0.5

This release fixes a few minor TS issues.

What's Changed

• Provider: pass state (S) generic through to ProviderProps by @​OliverJAsh in reduxjs/react-redux#1960
• wrap equalityFn type in NoInfer by @​phryneas in reduxjs/react-redux#1965
• Fix wrapped component prop types when passing nullish mapDispatchToProps by @​marconi1992 in reduxjs/react-redux#1928

Full Changelog: reduxjs/react-redux@v8.0.4...v8.0.5

v8.0.4

This patch release fixes some minor TS types issues, and updates the rarely-used areStatesEqual option for connect to now pass through ownProps for additional use in determining which pieces of state to compare if desired.

Note: 8.0.3 was accidentally published without one of these fixes. Use 8.0.4 instead.

Changelog

TS Fixes

We've fixed an import of React that caused issues with the allowSyntheticDefaultImports TS compiler flag in user projects.

connect already accepted a custom context instance as props.context, and had runtime checks in case users were passing through a real value with app data as props.context instead. However, the TS types did not handle that case, and this would fail to compile. If your own component expects props.context with actual data, connect's types now use that type instead.

The ConnectedProps<T> type had a mismatch with React's built-in React.ComponentProps<Component> type, and that should now work correctly.

Other Changes

The areStatesEqual option to connect now receives ownProps as well, in case you need to make a more specific comparison with certain sections of state.

The new signature is:

{
  areStatesEqual?: (
    nextState: State,
    prevState: State,
    nextOwnProps: TOwnProps,
    prevOwnProps: TOwnProps
  ) => boolean
}

What's Changed

• Don't require allowSyntheticDefaultImports: true by @​apepper in reduxjs/react-redux#1924
• Fixed type issue with ComponentProps from older @types/react by @​Andarist in reduxjs/react-redux#1956
• connect: pass ownProps to areStatesEqual by @​jspurlin in reduxjs/react-redux#1951
• Omit built-in context prop if user component props include context by @​markerikson in reduxjs/react-redux#1958

... (truncated)

Commits

• 8d03182 Release 8.0.5
• 32e40e4 Merge pull request #1928 from marconi1992/fix/1927
• ff7d96b fix: type when passing nullish mapDispathToProps
• 5082af7 Merge pull request #1965 from reduxjs/draft/amazing-rosalind
• bafe55e Add typetests
• fc3954a Update test matrix to Node 16 and TS 4.9
• 401250e also update TypedUseSelectorHook
• 3b06061 wrap type in
• ad40b97 Merge pull request #1960 from OliverJAsh/patch-1
• 8cf538c Merge pull request #1966 from ethen001/chores/custom-equality-fn-docs
• Additional commits viewable in compare view

Dependabot will resolve any conflicts with this PR as long as you don't alter it yourself. You can also trigger a rebase manually by commenting @dependabot rebase.

---

Dependabot commands and options

You can trigger Dependabot actions by commenting on this PR:

• @dependabot rebase will rebase this PR
• @dependabot recreate will recreate this PR, overwriting any edits that have been made to it
• @dependabot merge will merge this PR after your CI passes on it
• @dependabot squash and merge will squash and merge this PR after your CI passes on it
• @dependabot cancel merge will cancel a previously requested merge and block automerging
• @dependabot reopen will reopen this PR if it is closed
• @dependabot close will close this PR and stop Dependabot recreating it. You can achieve the same result by closing it manually
• @dependabot ignore this major version will close this PR and stop Dependabot creating any more for this major version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this minor version will close this PR and stop Dependabot creating any more for this minor version (unless you reopen the PR or upgrade to it yourself)
• @dependabot ignore this dependency will close this PR and stop Dependabot creating any more for this dependency (unless you reopen the PR or upgrade to it yourself)

[mgrabovsky]

This is going to need a couple of changes in our code. I've started with crafting a patch, but I'll leave this blocked for now.

[mgrabovsky]

I've updated the affected code and converted our reducers to use Redux Toolkit. However, the gating tests page reducer seems to have got broken, so I'm blocking this for now.

[Andrei-Stepanov]

In general I feel strange about this PR.
Was it necessary to rename reducer to slice in this PR?
Was this term slice introduced in this major release update?
Also see note about removing actions.

I think, that in this PR should be code that resolves update redux from 7 to 8

[Andrei-Stepanov]

Created a jira-ticket for this: https://issues.redhat.com/browse/OSCI-4547

[mgrabovsky]

@Andrei-Stepanov I think this is ready for review. I've split the changes so that there's one commit per slice/reducers, so it will be easier to review going through the commits one by one.

[Andrei-Stepanov]

Will this actually work? Without update ?

import update from 'immutability-helper';

[mgrabovsky]

Yes, Redux Toolkit uses the Immer library to handle in-place state updates. So we can now write code like in the old times, but it's still sort of immutable in the background.

[Andrei-Stepanov]

@mgrabovsky looks good.
But I have one question about import update from 'immutability-helper';
You removed. I am not sure that it is correct.
Could you please explain this place a bit more.
Purpose of this utility is to updata object at specified path, using a specific method, creating a NEW object with old values and new values at specified path.
How new code works at this place?

[mgrabovsky]

@Andrei-Stepanov See my response above. Redux Toolkit uses Immer under the hood, which watches for mutations on the state and automatically creates an immutable copy with the changes applied in the background.

This means that we don't have to call update() in Redux reducers any more and we can instead modify the state "directly", as Immer will do the updates for us automatically.

[mgrabovsky]

I have now added the clarifying comment as requested above.

[Andrei-Stepanov]

I have now added the clarifying comment as requested above.

Thank you! Please merge.