Skip to content

Commit

Permalink
Merge pull request #10818 from DestinyItemManager/react-router
Browse files Browse the repository at this point in the history
Upgrade to React Router v7
  • Loading branch information
bhollis authored Dec 8, 2024
2 parents 24d42c1 + e7e6053 commit 19605c9
Show file tree
Hide file tree
Showing 21 changed files with 57 additions and 51 deletions.
3 changes: 1 addition & 2 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -231,8 +231,7 @@
"react-dom": "^18.3.1",
"react-dropzone": "^14.3.5",
"react-redux": "^9.1.2",
"react-router": "^6.28.0",
"react-router-dom": "^6.28.0",
"react-router": "^7.0.1",
"react-textarea-autosize": "^8.5.5",
"redux": "^5.0.1",
"redux-thunk": "^3.1.0",
Expand Down
58 changes: 31 additions & 27 deletions pnpm-lock.yaml

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

8 changes: 6 additions & 2 deletions src/app/App.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -136,8 +136,12 @@ export default function App() {
</ErrorBoundary>
}
/>
<Route path="armory/*" element={<DefaultAccount />} />
<Route path=":membershipId/:destinyVersion/*" element={<Destiny />} />
<Route path="armory">
<Route path="*" element={<DefaultAccount />} />
</Route>
<Route path=":membershipId/:destinyVersion">
<Route path="*" element={<Destiny />} />
</Route>
<Route path="*" element={<DefaultAccount />} />
</>
)}
Expand Down
4 changes: 2 additions & 2 deletions src/app/Root.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import {
} from 'react-dnd-multi-backend';
import { TouchBackend } from 'react-dnd-touch-backend';
import { Provider } from 'react-redux';
import { BrowserRouter as Router } from 'react-router-dom';
import { BrowserRouter as Router } from 'react-router';
import App from './App';
import store from './store/store';
import { isNativeDragAndDropSupported } from './utils/browsers';
Expand Down Expand Up @@ -37,7 +37,7 @@ function Root() {
],
};
return (
<Router future={{ v7_startTransition: true }} basename={$PUBLIC_PATH}>
<Router basename={$PUBLIC_PATH}>
<Provider store={store}>
<LocationSwitcher />
<DndProvider options={options}>
Expand Down
2 changes: 1 addition & 1 deletion src/app/accounts/MenuAccounts.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { useThunkDispatch } from 'app/store/thunk-dispatch';
import { chainComparator, compareBy, reverseComparator } from 'app/utils/comparators';
import React from 'react';
import { useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import { AppIcon, signOutIcon } from '../shell/icons';
import Account from './Account';
import styles from './MenuAccounts.m.scss';
Expand Down
2 changes: 1 addition & 1 deletion src/app/accounts/SelectAccount.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ import { AppIcon, signOutIcon } from 'app/shell/icons';
import { useThunkDispatch } from 'app/store/thunk-dispatch';
import { chainComparator, compareBy, reverseComparator } from 'app/utils/comparators';
import { useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import Account from './Account';
import styles from './SelectAccount.m.scss';
import { logOut } from './platforms';
Expand Down
2 changes: 1 addition & 1 deletion src/app/compare/Compare.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,7 @@ import { StatHashes } from 'data/d2/generated-enums';
import { maxBy } from 'es-toolkit';
import React, { useCallback, useEffect, useMemo, useState } from 'react';
import { useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import Sheet from '../dim-ui/Sheet';
import { DimItem, DimSocket } from '../inventory/item-types';
import { chainComparator, Comparator, compareBy, reverseComparator } from '../utils/comparators';
Expand Down
2 changes: 1 addition & 1 deletion src/app/loadout/Loadouts.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,7 @@ import { usePageTitle } from 'app/utils/hooks';
import { DestinySeasonDefinition } from 'bungie-api-ts/destiny2';
import { useCallback, useEffect, useMemo, useRef, useState } from 'react';
import { useDispatch, useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import styles from './Loadouts.m.scss';
import LoadoutRow from './LoadoutsRow';
import { updateLoadoutStore } from './actions';
Expand Down
2 changes: 1 addition & 1 deletion src/app/loadout/loadout-menu/LoadoutPopup.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -62,7 +62,7 @@ import clsx from 'clsx';
import consumablesIcon from 'destiny-icons/general/consumables.svg';
import React, { useState } from 'react';
import { useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import { InGameLoadoutIconWithIndex } from '../ingame/InGameLoadoutIcon';
import { applyInGameLoadout } from '../ingame/ingame-loadout-apply';
import { inGameLoadoutsForCharacterSelector } from '../ingame/selectors';
Expand Down
2 changes: 1 addition & 1 deletion src/app/loadout/loadout-ui/OptimizerButton.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -5,7 +5,7 @@ import { Loadout, ResolvedLoadoutItem } from 'app/loadout/loadout-types';
import { AppIcon, faCalculator } from 'app/shell/icons';
import { count } from 'app/utils/collections';
import { useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';

/**
* Link to open a loadout in the Optimizer.
Expand Down
2 changes: 1 addition & 1 deletion src/app/login/Login.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ import { exportBackupData, exportLocalData } from 'app/storage/export-data';
import { useThunkDispatch } from 'app/store/thunk-dispatch';
import { isAppStoreVersion } from 'app/utils/browsers';
import { useEffect, useMemo, useState } from 'react';
import { Link, useLocation } from 'react-router-dom';
import { Link, useLocation } from 'react-router';
import { oauthClientId } from '../bungie-api/bungie-api-utils';
import styles from './Login.m.scss';

Expand Down
2 changes: 1 addition & 1 deletion src/app/records/Records.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,7 @@ import { querySelector } from 'app/shell/selectors';
import { compact, filterMap } from 'app/utils/collections';
import { usePageTitle } from 'app/utils/hooks';
import { useSelector } from 'react-redux';
import { useSearchParams } from 'react-router-dom';
import { useSearchParams } from 'react-router';
import { DestinyAccount } from '../accounts/destiny-account';
import ErrorBoundary from '../dim-ui/ErrorBoundary';
import {
Expand Down
2 changes: 1 addition & 1 deletion src/app/shell/About.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ import { usePageTitle } from 'app/utils/hooks';
import { systemInfo } from 'app/utils/system-info';
import logo from 'images/dimlogo.svg';
import { useEffect } from 'react';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import ExternalLink from '../dim-ui/ExternalLink';
import styles from './About.m.scss';
import { AppIcon, faDiscord, faGithub, faTshirt, heartIcon, helpIcon, mastodonIcon } from './icons';
Expand Down
2 changes: 1 addition & 1 deletion src/app/shell/GATracker.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { gaPageView } from 'app/google';
import { useEffect } from 'react';
import { useLocation } from 'react-router-dom';
import { useLocation } from 'react-router';

/**
* Record page views to Google Analytics.
Expand Down
3 changes: 1 addition & 2 deletions src/app/shell/Header.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -20,8 +20,7 @@ import { AnimatePresence, Spring, Variants, motion } from 'framer-motion';
import logo from 'images/logo-type-right-light.svg';
import React, { Suspense, lazy, useCallback, useEffect, useMemo, useRef, useState } from 'react';
import { useSelector } from 'react-redux';
import { useLocation } from 'react-router';
import { Link, NavLink } from 'react-router-dom';
import { Link, NavLink, useLocation } from 'react-router';
import { useSubscription } from 'use-subscription';
import ClickOutside from '../dim-ui/ClickOutside';
import ExternalLink from '../dim-ui/ExternalLink';
Expand Down
2 changes: 1 addition & 1 deletion src/app/shell/LocationSwitcher.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { resetRouterLocation } from 'app/shell/actions';
import { routerLocationSelector } from 'app/shell/selectors';
import { useEffect } from 'react';
import { useDispatch, useSelector } from 'react-redux';
import { useNavigate } from 'react-router-dom';
import { useNavigate } from 'react-router';

/**
* Component used to auto-navigate to a selected route selected through
Expand Down
2 changes: 1 addition & 1 deletion src/app/shell/Privacy.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import ExternalLink from 'app/dim-ui/ExternalLink';
import StaticPage from 'app/dim-ui/StaticPage';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import styles from './Privacy.m.scss';

export default function Privacy() {
Expand Down
2 changes: 1 addition & 1 deletion src/app/shell/ScrollToTop.tsx
Original file line number Diff line number Diff line change
@@ -1,5 +1,5 @@
import { useEffect } from 'react';
import { useLocation } from 'react-router-dom';
import { useLocation } from 'react-router';

/**
* https://reacttraining.com/react-router/web/guides/scroll-restoration
Expand Down
2 changes: 1 addition & 1 deletion src/app/shell/SneakyUpdates.tsx
Original file line number Diff line number Diff line change
@@ -1,6 +1,6 @@
import { dimNeedsUpdate$, reloadDIM } from 'app/register-service-worker';
import { useEffect, useRef } from 'react';
import { useLocation } from 'react-router-dom';
import { useLocation } from 'react-router';

/**
* "Sneaky Updates" - reload on navigation if DIM needs an update.
Expand Down
2 changes: 1 addition & 1 deletion src/app/storage/DimApiSettings.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ import { useThunkDispatch } from 'app/store/thunk-dispatch';
import { errorMessage } from 'app/utils/errors';
import React, { useState } from 'react';
import { useSelector } from 'react-redux';
import { Link } from 'react-router-dom';
import { Link } from 'react-router';
import styles from './DimApiSettings.m.scss';
import ImportExport from './ImportExport';
import LocalStorageInfo from './LocalStorageInfo';
Expand Down
2 changes: 1 addition & 1 deletion src/app/whats-new/WhatsNewLink.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ import { t } from 'app/i18next-t';
import { bungieAlertsSelector } from 'app/shell/selectors';
import clsx from 'clsx';
import { useSelector } from 'react-redux';
import { NavLink } from 'react-router-dom';
import { NavLink } from 'react-router';
import { useSubscription } from 'use-subscription';
import { dimNeedsUpdate$, reloadDIM } from '../register-service-worker';
import { AppIcon, updateIcon } from '../shell/icons';
Expand Down

0 comments on commit 19605c9

Please sign in to comment.