fix(routing): handle empty boardPath and resolve directory codes

This commit is contained in:
plebeius
2025-11-07 18:54:45 +01:00
parent 128b6b1986
commit b886cbdf3b
3 changed files with 41 additions and 25 deletions
+12 -2
View File
@@ -45,7 +45,12 @@ const CatalogButton = ({ address, isInAllView, isInSubscriptionsView, isInModVie
if (params?.timeFilterName) return `/mod/catalog/${params.timeFilterName}`; if (params?.timeFilterName) return `/mod/catalog/${params.timeFilterName}`;
return `/mod/catalog`; return `/mod/catalog`;
} }
const boardPath = address ? getBoardPath(address, defaultSubplebbits) : defaultSubplebbits || ''; let boardPath = '';
if (address) {
boardPath = getBoardPath(address, defaultSubplebbits);
} else if (Array.isArray(defaultSubplebbits) && defaultSubplebbits.length > 0 && defaultSubplebbits[0]?.address) {
boardPath = getBoardPath(defaultSubplebbits[0].address, defaultSubplebbits);
}
return `/${boardPath}/catalog`; return `/${boardPath}/catalog`;
}; };
@@ -83,7 +88,12 @@ const ReturnButton = ({ address, isInAllView, isInSubscriptionsView, isInModView
if (params?.timeFilterName) return `/mod/${params.timeFilterName}`; if (params?.timeFilterName) return `/mod/${params.timeFilterName}`;
return `/mod`; return `/mod`;
} }
const boardPath = address ? getBoardPath(address, defaultSubplebbits) : defaultSubplebbits || ''; let boardPath = '';
if (address) {
boardPath = getBoardPath(address, defaultSubplebbits);
} else if (Array.isArray(defaultSubplebbits) && defaultSubplebbits.length > 0 && defaultSubplebbits[0]?.address) {
boardPath = getBoardPath(defaultSubplebbits[0].address, defaultSubplebbits);
}
return `/${boardPath}`; return `/${boardPath}`;
}; };
+24 -21
View File
@@ -190,27 +190,30 @@ const Board = () => {
(isInAllView || isInSubscriptionsView || isInModView) && (isInAllView || isInSubscriptionsView || isInModView) &&
showMorePostsSuggestion && showMorePostsSuggestion &&
monthlyFeed.length > feed.length && monthlyFeed.length > feed.length &&
(weeklyFeed.length > feed.length ? ( (() => {
<div className={styles.morePostsSuggestion}> const basePath = isInAllView ? '/all' : isInSubscriptionsView ? '/subscriptions' : isInModView ? '/mod' : boardPath ? `/${boardPath}` : '';
<Trans return weeklyFeed.length > feed.length ? (
i18nKey='more_threads_last_week' <div className={styles.morePostsSuggestion}>
values={{ currentTimeFilterName, count: feed.length }} <Trans
components={{ i18nKey='more_threads_last_week'
1: <Link to={(isInAllView ? '/all' : isInSubscriptionsView ? '/subscriptions' : isInModView ? '/mod' : `/${boardPath}`) + '/1w'} />, values={{ currentTimeFilterName, count: feed.length }}
}} components={{
/> 1: <Link to={`${basePath}/1w`} />,
</div> }}
) : ( />
<div className={styles.morePostsSuggestion}> </div>
<Trans ) : (
i18nKey='more_threads_last_month' <div className={styles.morePostsSuggestion}>
values={{ currentTimeFilterName, count: feed.length }} <Trans
components={{ i18nKey='more_threads_last_month'
1: <Link to={(isInAllView ? '/all' : isInSubscriptionsView ? '/subscriptions' : isInModView ? '/mod' : `/${boardPath}`) + '/1m'} />, values={{ currentTimeFilterName, count: feed.length }}
}} components={{
/> 1: <Link to={`${basePath}/1m`} />,
</div> }}
)) />
</div>
);
})()
)} )}
</> </>
); );
+5 -2
View File
@@ -1,6 +1,8 @@
import { useState } from 'react'; import { useState } from 'react';
import { Link, useLocation } from 'react-router-dom'; import { Link, useLocation } from 'react-router-dom';
import useSubplebbitsStore from '@plebbit/plebbit-react-hooks/dist/stores/subplebbits'; import useSubplebbitsStore from '@plebbit/plebbit-react-hooks/dist/stores/subplebbits';
import { useDefaultSubplebbits } from '../../hooks/use-default-subplebbits';
import { getSubplebbitAddress } from '../../lib/utils/route-utils';
import { HomeLogo } from '../home'; import { HomeLogo } from '../home';
import styles from './not-found.module.css'; import styles from './not-found.module.css';
import { NOT_FOUND_IMAGES } from '../../generated/asset-manifest'; import { NOT_FOUND_IMAGES } from '../../generated/asset-manifest';
@@ -16,7 +18,8 @@ const NotFound = () => {
// Extract boardIdentifier from pathname (could be directory code or address) // Extract boardIdentifier from pathname (could be directory code or address)
const pathParts = location.pathname.split('/').filter(Boolean); const pathParts = location.pathname.split('/').filter(Boolean);
const boardIdentifier = pathParts[0] && pathParts[0] !== 'not-found' && pathParts[0] !== 'faq' ? pathParts[0] : ''; const boardIdentifier = pathParts[0] && pathParts[0] !== 'not-found' && pathParts[0] !== 'faq' ? pathParts[0] : '';
const subplebbitAddress = boardIdentifier || ''; const defaultSubplebbits = useDefaultSubplebbits();
const subplebbitAddress = boardIdentifier ? getSubplebbitAddress(boardIdentifier, defaultSubplebbits) : '';
const subplebbit = useSubplebbitsStore((state) => state.subplebbits[subplebbitAddress]); const subplebbit = useSubplebbitsStore((state) => state.subplebbits[subplebbitAddress]);
const { address, shortAddress } = subplebbit || {}; const { address, shortAddress } = subplebbit || {};
@@ -35,7 +38,7 @@ const NotFound = () => {
<> <>
<br /> <br />
<div className={styles.backToBoard}> <div className={styles.backToBoard}>
[<Link to={`/${boardIdentifier || subplebbitAddress}`}>Back to p/{shortAddress}</Link>] [<Link to={`/${boardIdentifier || subplebbitAddress}`}>Back to p/{shortAddress || subplebbitAddress}</Link>]
</div> </div>
</> </>
)} )}