-
Notifications
You must be signed in to change notification settings - Fork 585
Open
Labels
Description
Some functionality has been deprecated in that library, we need to understand what the replacement is, and whether it is compatible with the modern React at all.
Per @anshgoyalevil's comment #2531 (comment)
The steps towards upgrading the project as a whole would be to
- First shift to functional components in place of class components
- [Feature]: Refactor class components to functional components #2610
- NB: that issue doesn't need to be addressed 100%, only for files that use
history
, in order to make progress here
- After that is done, try removing the instances of the
history
package and useuseNavigate
hook (you can now use hooks because functional components are allowed to do that) - The project already uses
react-router-dom
v5 compat v6, so you can directly usereact-router-dom
v6 APIs without breaking the whole project. Aim for 1 file/router at a time in one PR.
Parship999