Skip to content

Refactor away custom Route to a wrapper component instead

Yuto Takano requested to merge remove-userroutes into master

I'm looking to upgrade react-router from v5 to v7.

As part of that, this MR future-proofs how we implement authenticated routes. Currently we create a replacement route component called UserRoute and accept all props that Route would accept.

React-Router v6/7 have removed support for this method. Instead, it advocates for implementing authenticated routes as a wrapper component that houses multiple Route inside. This MR does that.

Merge request reports