Загрузка...

API Исправлено Неправильная пагинация в Lolzteam API

Тема в разделе Рассмотренные недочеты создана пользователем its_niks 22 апр 2025. 251 просмотр

  1. its_niks
    Кому понравилось (2)
    its_niks Автор темы 22 апр 2025 make testers great again! 16 934 29 янв 2021
    Скриншот/видео недочета: [IMG]
    Полная ссылка на страницу, где возникает проблема: https://prod-api.zelenka.guru/posts?thread_id=3938630&page=12
    Как воспроизвести недочет: Как видите в пагинации показывает 3 страницы, но по факту их 12, я так понимаю это из-за того что начали недавно отображаться свои удаленные сообщения. Считаю можно добавить параметр, который отключает возврат своих удаленных сообщений. Ну или правильно отображать пагинацию с учетом их.
     
    22 апр 2025
    1. Посмотреть предыдущие комментарии (2)
    2. its_niks Автор темы
      AS7RID, думаю можно параметр include_deleted_posts добавить. Если с ним запрос делать,, то в пагинации в данном случае будет 12 страниц, если без - 3. Все же думаю не хорошо, что ответ апи напрямую зависит от настроек на сайте
      22 апр 2025 Изменено
    3. xellan
      AS7RID, пользователи не должны знать о удаленных сообщениях и пагинация не должна быть сбитой. Если пропускать удаленные смс, то на странице окажется меньше смс, такой логике нет на сайте. К примеру на первой странице 2 сообщения валидные и 18 удаленных, а на второй странице 20 сообщений как положено.

      Если считать в апишке удаленные смс и нет, страниц будет отображено больше чем на сам деле, если удаленных смс так много.
      Достаточно сортировать данные без удаленных смс чтобы страницы и сообщения соответствовали с данными сайта.
      Если по какой то причине в апи нужны удаленные смс, для модерации и т.п можно выдавать роль чтобы к этому был доступ и данные не сортировались.
      26 апр 2025
    4. WhatACat
      14 май 2025
  2. uncpfiae
    uncpfiae 25 май 2025 ку 10 931 29 сен 2023
    удаленные сообщения теперь всегда отображаются в апи, независимо от настройки
     
    25 май 2025
Top
Загрузка...