Ошибка 404 в "Что нового?"
Находясь на странице https://typescript-definitive-guide.ru/what-is-new/[version]/ на самой старой и на самой новой записи в списке changelog'ов версий, при нажатии на кнопки "Предыдущая глава" и "Следующая глава" соответственно, кнопка ведёт на несуществующий путь [version], что, очевидно, приводит к 404 error.
Проблема в этих строчках (возможно, нужно добавить какой-то дизейбл к next/link для крайних записей (поправил бы сам, но не умею в react и next 😅) https://github.com/nauchikus/typescript-definitive-guide/blob/580ce0a3fe734454b42a8847e7aaf48bfdb99613/www/pages/what-is-new/%5Bversion%5D.tsx#L137 https://github.com/nauchikus/typescript-definitive-guide/blob/580ce0a3fe734454b42a8847e7aaf48bfdb99613/www/pages/what-is-new/%5Bversion%5D.tsx#L192
P.S.: небольшая рекомендация - в этом же файле для типа PageNavInfo https://github.com/nauchikus/typescript-definitive-guide/blob/580ce0a3fe734454b42a8847e7aaf48bfdb99613/www/pages/what-is-new/%5Bversion%5D.tsx#L57-L63 Можно было бы переименовать переменные isPrevPage и isNextPage на hasPrevPage и hasNextPage для большей семантичности
P.P.S.: проект огонь, огромная благодарочка, автор, с меня звёздочка и чеканная монета)
@dsvitnev Спасибо! Прямо сейчас просто отключил события, но в скором будущем обязательно исправлю более приемлемо.
С замечанием по поводу has тоже согласен. Я тоже на этот когда-то внимание обращал, но потом отвык. В js мире вообще не встречал has или can. Все проверки только на is.
@nauchikus Да, без проблем) Ту же проблему обнаружил и в pages/book/chapters/[chapterId].tsx, на первой и последней главах в книжке кнопки также ведут вникуда на роут /[chapterId] и 404 ошибку соответственно https://github.com/nauchikus/typescript-definitive-guide/blob/6b7dc7fda26de8acb4698e17a9d101751bfb6edf/www/pages/book/chapters/%5BchapterId%5D.tsx#L140 https://github.com/nauchikus/typescript-definitive-guide/blob/6b7dc7fda26de8acb4698e17a9d101751bfb6edf/www/pages/book/chapters/%5BchapterId%5D.tsx#L194
UPD. В "Что нового?" кнопка "Следующая глава" перестала работать (это следствие ad-hoc "прямо сейчас просто отключил события"?), а сценарий с кнопкой "Предыдущая глава" и самой старой публикацией в разделе по прежнему остался таким же (ведёт на 404) В общем, успехов в починке 😉✌️