Пользователи жалуются на потерю очков

Victor_Bramante · 21.Декабрь.2024 16:10:03

Привет, я пытаюсь разобраться, как это работает.

Пытался настроить 2 таблицы лидеров для определения победителей ноября 2024 года и декабря 2024 года. Установил фиксированные даты для обоих периодов (то есть с 01.11 по 30.11), но ни один из них не работает корректно, потому что:

вас заставляют использовать период по умолчанию, даже если указано, что это опционально

default period option set to "All Time"307×105 2.4 KB
если выбрать «Ежемесячно», подсчитываются очки за последние 30 дней, даже при наличии фиксированных дат
если выбрать «Всегда», суммируются все очки, и фиксированные даты игнорируются

Что мне нужно сделать, чтобы подсчитать очки за ноябрь 2024 года? Пользователи видят, что теряют очки каждый день, поэтому пишут мне с жалобами.

Также было бы здорово добавить возможность скрыть селектор в публичном виде таблицы лидеров.

selase · 30.Декабрь.2024 19:04:05

Оставление периода по умолчанию как Все время должно работать корректно. Здесь Все время подразумевается в пределах установленных дат «с»/«по» для таблицы лидеров, если они заданы.

Однако, похоже, существует ошибка: фильтрация таблицы лидеров по определённым периодам даёт некорректные результаты. Реализация score_period_condition не учитывает должным образом даты «с»/«по» таблицы лидеров и вступает в конфликт с основной проверкой границ таблицы лидеров.

github.com/discourse/discourse-gamification

lib/discourse_gamification/leaderboard_cached_view.rb

03ea20da0


      
          def score_period_condtion(period)
            date =
              case period
              when "yearly"
                "CURRENT_DATE - INTERVAL '1 year'"
              when "monthly"
                "CURRENT_DATE - INTERVAL '1 month'"
              when "quarterly"
                "CURRENT_DATE - INTERVAL '3 month'"
              when "weekly"
                "CURRENT_DATE - INTERVAL '1 week'"
              when "daily"
                "CURRENT_DATE - INTERVAL '1 day'"
              else
                nil
              end
          
            date ? "AND gs.date >= #{date}" : ""
          end

github.com/discourse/discourse-gamification

lib/discourse_gamification/leaderboard_cached_view.rb

03ea20da0


      
          WHERE
            (
              -- Leaderboard with both "to/from" dates
              -- Only scores created between specified dates
              (
                lb.from_date IS NOT NULL
                AND lb.to_date IS NOT NULL
                AND gs.date BETWEEN lb.from_date AND lb.to_date
              )
              OR
              -- Leaderboard without "from/to" dates. All scores
              (lb.from_date IS NULL AND lb.to_date IS NULL)
              OR
              -- Leaderboard with just "from" date
              -- Only scores created starting from the specified date
              (
                lb.from_date IS NOT NULL
                AND lb.to_date IS NULL
                AND gs.date >= lb.from_date
              )

This file has been truncated. show original

Я занимаюсь этим вопросом, исправление должно быть готово в ближайшее время.

Victor_Bramante · 30.Декабрь.2024 19:51:56

Спасибо, дай знать, когда сделаешь, чтобы я мог проверить снова.

С Новым годом!

Victor_Bramante · 13.Январь.2025 23:19:52

Привет!

Было ли применено исправление? Похоже, что мои лидерборды за декабрь показывают некорректные результаты.

selase · 14.Январь.2025 11:30:22

Приносим извинения за задержку. Исправление ещё не применено, но у меня есть два ожидающих рассмотрения PR. Обновлю информацию после их слияния.

FIX: Redo period filtering to respect leaderboard date bounds by s3lase · Pull Request #178 · discourse/discourse-gamification · GitHub исправляет запрос материализованного представления.
FEATURE: Add option to lock leaderboard view to default period by s3lase · Pull Request #179 · discourse/discourse-gamification · GitHub добавляет опцию отключения селектора периода времени.

selase · 20.Январь.2025 12:41:47

Спасибо за ваше терпение. Изменения были объединены и теперь доступны для проверки. Пожалуйста, сообщите нам, если у вас возникнут дополнительные проблемы или вопросы.

Тема		Ответов	Просм.
Gamification leaderboard pagination doesn't respect the selected date period Bug gamification	2	569	04.01.2023
Setting a leaderboard date range should lock the default period Feature gamification	4	524	22.01.2025
Gamification period selector constraints UX gamification	1	370	09.03.2023
Leaderboard doesn't show score for recent periods Support gamification	5	388	12.01.2023
How to create a new leaderboard for a specific month? Support gamification	3	559	12.01.2023

Пользователи жалуются на потерю очков

Связанные темы