dotkom / AMaCS

Committee application application for Online online.
5 stars 2 forks source link

Move away from class based components #117

Closed oleast closed 4 years ago

oleast commented 4 years ago

Is your feature request related to a problem? Please describe. We use unsafe lifecycle hooks and all of that. Maybe move away from classes altogether. Make us ready for concurrent mode and all of that.

Describe the solution you'd like Move Application form state from local state to a reducer/slice from Redux Toolkit, that is the most crucial class component. The rest should hopefully be trivial to convert.

Describe alternatives you've considered Not upgrading would be bad in the long run.

Additional context Concurrent mode is still en experimental feature of React, and it won't be enforced even when it is released. But having it would still be cool.