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.

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

feat!: major version upgrade of react-router-version to v6 #422

Merged
merged 9 commits into from
Aug 8, 2023

Conversation

Mashal-m
Copy link
Contributor

@Mashal-m Mashal-m commented Dec 28, 2022

Ticket

Upgrade React Router to v6.

BREAKING CHANGES

  • Upgrade react-router-dom to v6. (Guideline to migrate from v5 to v6)

Copy link
Contributor

@arbrandes arbrandes left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Untested, but can't find a problem with the code. However, how will this affect downstream? Will MFEs have to update their route invocations as well?

@codecov
Copy link

codecov bot commented Mar 14, 2023

Codecov Report

Patch coverage: 100.00% and project coverage change: +0.07% 🎉

Comparison is base (bc369e8) 82.98% compared to head (536df86) 83.05%.
Report is 39 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #422      +/-   ##
==========================================
+ Coverage   82.98%   83.05%   +0.07%     
==========================================
  Files          40       40              
  Lines        1040     1062      +22     
  Branches      187      195       +8     
==========================================
+ Hits          863      882      +19     
- Misses        165      168       +3     
  Partials       12       12              
Files Changed Coverage Δ
src/initialize.js 98.78% <ø> (+0.18%) ⬆️
src/react/AppProvider.jsx 83.33% <100.00%> (+3.33%) ⬆️
src/react/AuthenticatedPageRoute.jsx 100.00% <100.00%> (ø)
src/react/PageWrap.jsx 100.00% <100.00%> (ø)

... and 5 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@abdullahwaheed abdullahwaheed changed the title build: major version upgrade of react-router-version upgrade feat!: major version upgrade of react-router-version to v6 Mar 14, 2023
@Syed-Ali-Abbas-Zaidi Syed-Ali-Abbas-Zaidi force-pushed the mashal-m/react-router_version_upgrade branch from 85c90d8 to 6cb9d7e Compare March 14, 2023 10:09
@arbrandes
Copy link
Contributor

Let me know when this is ready for a final test/review.

@abdullahwaheed
Copy link
Contributor

@arbrandes its ready to review now

example/index.jsx Outdated Show resolved Hide resolved
@abdullahwaheed abdullahwaheed merged commit df7ed59 into master Aug 8, 2023
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants