Skip to content

Commit 59b70db

Browse files
authored
Merge pull request #1138 from ryanfitton/main
Fix for regex trialing slash if `trailingSlash: true`
2 parents de6326a + 9a8ffc1 commit 59b70db

File tree

3 files changed

+5
-4
lines changed

3 files changed

+5
-4
lines changed

layouts/ListLayout.tsx

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -26,7 +26,8 @@ function Pagination({ totalPages, currentPage }: PaginationProps) {
2626
const lastSegment = segments[segments.length - 1]
2727
const basePath = pathname
2828
.replace(/^\//, '') // Remove leading slash
29-
.replace(/\/page\/\d+$/, '') // Remove any trailing /page
29+
.replace(/\/page\/\d+\/?$/, '') // Remove any trailing /page
30+
.replace(/\/$/, '') // Remove trailing slash
3031
const prevPage = currentPage - 1 > 0
3132
const nextPage = currentPage + 1 <= totalPages
3233

layouts/ListLayoutWithTags.tsx

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -27,9 +27,8 @@ function Pagination({ totalPages, currentPage }: PaginationProps) {
2727
const lastSegment = segments[segments.length - 1]
2828
const basePath = pathname
2929
.replace(/^\//, '') // Remove leading slash
30-
.replace(/\/page\/\d+$/, '') // Remove any trailing /page
31-
console.log(pathname)
32-
console.log(basePath)
30+
.replace(/\/page\/\d+\/?$/, '') // Remove any trailing /page
31+
.replace(/\/$/, '') // Remove trailing slash
3332
const prevPage = currentPage - 1 > 0
3433
const nextPage = currentPage + 1 <= totalPages
3534

next.config.js

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -67,6 +67,7 @@ module.exports = () => {
6767
output,
6868
basePath,
6969
reactStrictMode: true,
70+
trailingSlash: false,
7071
pageExtensions: ['ts', 'tsx', 'js', 'jsx', 'md', 'mdx'],
7172
eslint: {
7273
dirs: ['app', 'components', 'layouts', 'scripts'],

0 commit comments

Comments
 (0)