Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

CUMULUS-3862: Upgrade React Router to v6 #1148

Open
wants to merge 29 commits into
base: develop
Choose a base branch
from
Open

CUMULUS-3862: Upgrade React Router to v6 #1148

wants to merge 29 commits into from

Conversation

jjmccoy
Copy link
Member

@jjmccoy jjmccoy commented Sep 25, 2024 •
edited
Loading

Summary: React Router from v5 to v6

Addresses CUMULUS-3862: Develop amazing new feature

Changes

PR Checklist

  • Update CHANGELOG
  • Unit tests
  • Adhoc testing
  • Integration tests

@jjmccoy jjmccoy marked this pull request as ready for review September 27, 2024 17:21
jjmccoy added 16 commits October 7, 2024 14:22
@jjmccoy
establish new queryparams and useselector hooks
34b8594
@jjmccoy
rework nested routes with login and auth to react router v6
e4ce414
@jjmccoy
establish new UrlHelper HOC for query params
2eb453f
@jjmccoy
fix router and location for auth to home page
cc6b48d
@jjmccoy
consolidate HOC wrappers into one
f9cfced
@jjmccoy
parent pages params to state routed
271d4dc
@jjmccoy
initial routing updates for child pages
a5e31c3
@jjmccoy
link parent and child page routes for top navigation
e232f64
@jjmccoy
initial table config link and button params
7a2b8f8
@jjmccoy
refactor SortableTable and select table-configs
689dffe
@jjmccoy
more table routing for links and buttons
aa7e3ed
@jjmccoy
addtl routing updates
0ddab3f
@jjmccoy
more routing updates and console error fixes
c43285c
@jjmccoy
Merge branch 'develop' into CUMULUS-3862
34f4bfa
@jjmccoy
update changelog
3010f1d
@jjmccoy
update packages with --legacy-peer-deps for react router work
77ce9ab
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Work In Progress
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@jjmccoy