hexletguides.github.io

hexletguides.github.io copied to clipboard

Поправил нахождение предыдущего и следующего постов в апи. Можно тестировать )

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Comments	Updated (UTC)
hexletguides	✅ Ready (Inspect)	Visit Preview	💬 Add feedback	Mar 4, 2024 0:28am

Поправил. В апи есть функция getPublishedPosts() и getPostsList(). Вот в postsList есть сортировка скрытых элементов и через нее генерились пропсы для страниц. При этом в остальных местах использовалась функция getPublishedPosts() которая не совсем корректно отражает свое название т.к. она не только открытые посты генерит, а еще и скрытые. Судя по всему getPostsList была добавлена позже т.к. в апи везде используется getPublishedPosts, и где она используются висят проверки на скрытые элементы. Соответственно, у нас есть два варианта:

• добавить проверку на скрытый элемент в getPublishedPosts и ею заменить getPostsList, а также зачистить проверки на скрытые элементы в местах использования getPublishedPosts
• либо оставить как у меня получилось Какой вариант реализовать?

@Ledchig

Вот в postsList есть сортировка скрытых элементов

сортировка? Может фильтрация?)

Посты могут быть скрыты, если их нужно опубликовать в драфт режиме, например чтобы что-то править по ходу. ПОэтому давайте использовать getPublishedPosts - она должна давать список опубликованных постов, которые видны людям. А есть контекст почему была добавлена getPostsList? (пулл реквесты?)

@Ledchig Привет, как успехи? Нужна помощь?

@fey Привет. Нашел Posts locales #397 - вот такой пуллреквест. В нем был добавлен getPublishedPosts и getPostsList. Тогда в getPublishedPosts была фильтрация скрытых. Затем при пуллреквесте next.js post mdx render #408 сортировка была перемещена в getPostsList, а так же для рендера статей, вместо использования getPostsList который выдает отфильтрованные посты, стала использоваться функция getPublishedPosts в которой удалили фильтрацию. Такие вот итоги расследования ) Можем изменить имя getPublishedPosts на getPosts, а там где нам нужно я уже подставил getPostsList который выдает отфильтрованные статьи. Норм вариант?

Присылайте код - посмотрим!

@fey переименовал. Можно смотреть)

@Ledchig я изменения ревернтну. Дело в том, что сломались редиректы с en на ru, когда в en локали поста фактически нету и нам нужен просто редирект https://guides.hexlet.io/encoding/