Skip to content

refactor: Migrate <Routes /> to react-router v6 #35030

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

Closed
MajorLift wants to merge 26 commits into from
Closed

refactor: Migrate <Routes /> to react-router v6 #35030

Changes from 1 commit
Commits
Show all changes
26 commits
Select commit Hold shift + click to select a range
1f91b8c
Convert target files to typescript
MajorLift May 15, 2025
7b1bb31
Remove HOC container for `Routes`
MajorLift Jul 31, 2025
f470f09
Fix selectors
MajorLift Jul 31, 2025
d3a96f3
Convert `<Routes />` to functional component
MajorLift Jul 31, 2025
b2ddb91
Fix dispatch callbacks
MajorLift Aug 4, 2025
1c53c97
consolidate feature flag branch for `isLoadingShown` definition
MajorLift Aug 4, 2025
21e68fe
Use `component` prop for `Route` instances
MajorLift Aug 4, 2025
61733a0
Handle undefined case for `openExtensionInBrowser`
MajorLift Aug 4, 2025
5d38733
Add `component` to proptypes of `Route` wrapper components
MajorLift Aug 5, 2025
2b9015b
Restore `exact` props to routes
MajorLift Aug 5, 2025
d6e3bc0
undefined case handling for `getSelectedAccountTokensAcrossChains`
MajorLift Aug 5, 2025
7231ab7
Fix wrong path for `CreateAccountPage`
MajorLift Aug 7, 2025
831c40b
Better comments
MajorLift Aug 7, 2025
6134411
Fix incorrect selector output destructuring
MajorLift Aug 7, 2025
b1a9e0f
Merge branch 'main' into chore/react-router-v6/250515-convert-Routes-…
MajorLift Aug 11, 2025
2dd9d4f
Revert changes to selectors
MajorLift Aug 11, 2025
10aee98
Revert "Revert changes to selectors"
MajorLift Aug 11, 2025
5f2e795
Add effect cleanup for `history.listen` call
MajorLift Aug 11, 2025
c867278
Prevent `openExtensionInBrowser` from triggering on unrelated deps ch…
MajorLift Aug 11, 2025
29b0b66
minimize re-renders by splitting `useEffect` calls by deps
MajorLift Aug 11, 2025
6dbf846
Remove unused variable
MajorLift Aug 11, 2025
2c740d2
Revert change to `getSelectedAccountTokensAcrossChains` behavior and …
MajorLift Aug 11, 2025
2da9b28
Restore destructuring undefined handling
MajorLift Aug 11, 2025
aeeb1c6
Restore early exit to `getSelectedAccountTokensAcrossChains` but retu…
MajorLift Aug 11, 2025
0fbfdd3
Bump `react-router-dom` to v6
MajorLift Aug 12, 2025
096fe44
Replace `Switch` with `Routes`
MajorLift Aug 12, 2025
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Fix wrong path for CreateAccountPage
  • Loading branch information
@MajorLift
MajorLift committed Aug 7, 2025
commit 7231ab7e88c185949a3e0eaec9f6f46f7f6a2cf3
2 changes: 1 addition & 1 deletion ui/pages/routes/routes.component.tsx
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -278,7 +278,7 @@ const ConfirmationPage = mmLazy(
const CreateAccountPage = mmLazy(
() =>
import(
'../../components/multichain/create-account/create-account.tsx'
'../create-account/create-account.component.js'
) as unknown as Promise<DefaultExportComponent>,
);
const NftFullImage = mmLazy(
Expand Down
Loading