Refactor redirect for plans page to not require a react-router version upgrade #4790
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Checklist
Description
Original redirect pr required a react-router version upgrade which was throwing an
AwaitRenderStatus
reference error, see internal discussion: https://chat.kobotoolbox.org/#narrow/stream/4-Kobo-Dev/topic/2.2E023.2E52.20release/near/318048Notes
Reverting react-router version back to
6.14.2
and commenting out theauthLoader
function which was dependent on the upgrade. MovedPROJECTS_ROUTES
tojsapp/js/projects/routes.tsx
to prevent circular dependency errors and updated therequireAuth
component to redirect appropriately based on the user's login status.