Skip to content
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.

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

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

Open
dsvitnev opened this issue Sep 26, 2022 · 2 comments
Open

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

dsvitnev opened this issue Sep 26, 2022 · 2 comments

Comments

@dsvitnev
Copy link

Находясь на странице 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.: проект огонь, огромная благодарочка, автор, с меня звёздочка и чеканная монета)

@nauchikus
Copy link
Owner

nauchikus commented Sep 27, 2022

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

@dsvitnev
Copy link
Author

dsvitnev commented Sep 27, 2022

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

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

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

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants