Я открыл PR, чтобы разрешить анонимным пользователям ставить «лайки» постам. Это настройка сайта, поэтому администраторы могут включать или отключать эту функцию.
Можно ли попросить код-ревью?
Контекст: я являюсь частью организации 501(c)3 на Discourse, где учётные записи зарегистрированы на реальные имена. Мы поощряем анонимные обсуждения, так как некоторые люди хотят делиться личным опытом и обсуждать деликатные темы (проблемы психического здоровья, тяжёлые воспоминания и т. д.). Анонимные обсуждения оказались очень содержательными и полезными, а добавление функции анонимного «лайка» поможет нам оценить взаимодействие сообщества по темам, где люди могут сильно сопереживать или соглашаться, но не чувствовать себя комфортно, публикуя своё имя.
Не могли бы вы, пожалуйста, дать краткий совет, где вызывается иконка запрета? Я ещё раз посмотрел, но так и не смог до конца разобраться.
Из комментария на GitHub:
Привет @nattsw, спасибо за повторный ревью. Я немного запутался, где именно должен происходить запрет, так как post_undo_action_window_mins упоминается только в can_delete_post_action?, и метод корректно возвращает false, когда время post_undo_action_window_mins истекло.
Однако, несмотря на то, что этот метод корректно возвращает false после закрытия окна, символ запрета не появляется, но нет другого метода, который бы ссылался на post_undo_action_window_mins. Я понимаю, что can_delete_post_action? вызывается при загрузке страницы, но не могу найти, где именно вызывается сама иконка запрета.
Не могли бы вы указать на блок кода, который вызывает символ запрета? Спасибо!
Спасибо за подсказку! Это было очень полезно! Я отправил патч, вручную проверил поведение и добавил дополнительное покрытие тестами. Когда у вас будет возможность, готов к новому раунду ревью.
