nauchikus / typescript-definitive-guide

TypeScript: Definitive Guide (book and docs in one)

Home Page:https://typescript-definitive-guide.ru

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Ошибка 404 в "Что нового?"

dsvitnev opened this issue · comments

Находясь на странице https://typescript-definitive-guide.ru/what-is-new/[version]/ на самой старой и на самой новой записи в списке changelog'ов версий, при нажатии на кнопки "Предыдущая глава" и "Следующая глава" соответственно, кнопка ведёт на несуществующий путь [version], что, очевидно, приводит к 404 error.

самая старая запись в списке...

самая новая запись в списке...

...кнопки на обоих ведут на 404 :(

Проблема в этих строчках (возможно, нужно добавить какой-то дизейбл к next/link для крайних записей (поправил бы сам, но не умею в react и next 😅)

<Link href={ pageNav.prevPage.path }>

<Link href={ pageNav.nextPage.path }>

P.S.: небольшая рекомендация - в этом же файле для типа PageNavInfo

type PageNavInfo = {
isPrevPage: boolean;
isNextPage: boolean;
prevPage: PageNav;
nextPage: PageNav;
}

Можно было бы переименовать переменные isPrevPage и isNextPage на hasPrevPage и hasNextPage для большей семантичности

P.P.S.: проект огонь, огромная благодарочка, автор, с меня звёздочка и чеканная монета)

@dsvitnev Спасибо! Прямо сейчас просто отключил события, но в скором будущем обязательно исправлю более приемлемо.
С замечанием по поводу has тоже согласен. Я тоже на этот когда-то внимание обращал, но потом отвык. В js мире вообще не встречал has или can. Все проверки только на is.

@nauchikus Да, без проблем)
Ту же проблему обнаружил и в pages/book/chapters/[chapterId].tsx, на первой и последней главах в книжке кнопки также ведут вникуда на роут /[chapterId] и 404 ошибку соответственно

<Link href={pageNav.prevPage.path}>

<Link href={pageNav.nextPage.path}>

UPD. В "Что нового?" кнопка "Следующая глава" перестала работать (это следствие ad-hoc "прямо сейчас просто отключил события"?), а сценарий с кнопкой "Предыдущая глава" и самой старой публикацией в разделе по прежнему остался таким же (ведёт на 404)
В общем, успехов в починке 😉✌️