rdmo icon indicating copy to clipboard operation
rdmo copied to clipboard

Switch to redux toolkit from legacy redux

Open afuetterer opened this issue 1 year ago • 5 comments

Refs:

  • https://redux.js.org/usage/migrations/migrating-rtk-2
  • https://redux.js.org/usage/migrating-to-modern-redux

afuetterer avatar Dec 09 '23 07:12 afuetterer

I looked at redux toolkit when I started working on the new management. I thought it was to complicated for what I wanted to archive. Back then, I was not used to use hooks though. I like how the redux store is set up now, and I would rather not change it ...

Same for React Router btw. (Although am an not sure anymore if it was so clever to implement routing myself.

Another tool I was considering was redux-persist, but again, it seemed to much overhead and to complicated compared to the solution we now have.

Maybe we should reconsider this. I can have a look when I am working on the new interview next year.

jochenklar avatar Dec 09 '23 09:12 jochenklar

I just opened this issue here, because the dependabot wanted to update react and redux and everything started failing.

Which is of course a good thing, the basic frontend tests are working.

The referenced release notes in #869 state, that the redux authors intend users to switch to redux-toolkit.

I have no strong opinion about this at all.

afuetterer avatar Dec 09 '23 09:12 afuetterer

Alright, then future-Jochen needs to solve this! :cry:

jochenklar avatar Dec 09 '23 09:12 jochenklar

bump future-Jochen, maybe want to look into this again before more React parts are added to the code-base?

MyPyDavid avatar Mar 07 '24 15:03 MyPyDavid

I did look into it and I don't get the new way for our rather complex ajax calls. I will stick for the pattern in management for now. In the end this can be refactored later.

jochenklar avatar May 17 '24 16:05 jochenklar