Moin
13.Июль.2026 21:53:58
1
Некоторое время назад в плагин голосования по темам была добавлена функция запроса уведомлений
lindsey:
После того как пользователь проголосует за идею, Discourse спросит его, хочет ли он получать уведомления о новых сообщениях в этой теме. Если он согласится, уровень уведомлений для этой темы будет изменен на «Наблюдение», чтобы он мог отслеживать новую активность. Это изменение позволяет пользователям легко подписаться на обновления по темам, которые их интересуют.
И вы также
Я заметил, что функция уведомлений, похоже, не работает, когда отключена опция topic voting enable vote limits (включить ограничение голосов).
У меня сложилось впечатление, что меню скрывается, если нет необходимости показывать оставшееся количество голосов, что также блокирует функцию уведомлений:
Голоса ограничены:
Голоса не ограничены:
2 лайка
Спасибо за отчет @Moin
main ← fix-topic-voting-notification-prompt-vote-limits
merged 12:54PM - 15 Jul 26 UTC
When topic voting vote limits are disabled, the "Notify me about new posts"
prom… pt shown after voting never appeared. That prompt is part of the menu the
vote button opens, but the menu was only rendered when vote limits were
enabled — with limits off the button had no menu at all, so both the
notification prompt and the remove-vote action were unreachable.
The fix renders the menu for any signed-in user rather than gating it on vote
limits. The remaining-votes rows stay tied to limits being enabled, while
remove-vote and watch-topic show whenever the user has voted. Removing a vote
with limits disabled now goes through the menu, matching the behavior when
limits are enabled.
Reported at https://meta.discourse.org/t/disabling-vote-limits-seems-to-break-notification-prompt-in-topic-voting/407512
The second commit is follow-up housekeeping on the plugin, kept separate from
the fix: removing dead code and duplication in the vote components and
initializers, counting a user's votes with `COUNT` instead of loading every
row on each current-user serialization, and deleting orphaned i18n keys and a
route-map filename left over from the plugin's former name.
### Testing
- Added a system spec covering the notification prompt appearing after voting
with vote limits disabled.
- Existing topic-voting system and JS specs pass; the model spec still covers
the trust-level-0 lock via `reached_voting_limit?`.
2 лайка
Это будет исправлено в