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

Luke_Cousins · 21.Декабрь.2023 20:06:26

Установите таймер темы для её закрытия через x дней
Закройте тему вручную
Вы увидите строку, в которой всё ещё указано, что тема будет закрыта через x дней
Нажмите на значок корзины рядом с ней, и появится сообщение об ошибке:

Очевидно, это очень мелкая проблема. Я обожаю Discourse и всё, что вы делаете, но решил, что стоит упомянуть об этом Спасибо

sam · 22.Декабрь.2023 02:24:01

Спасибо, что подняли этот вопрос. Я добавил тег pr-welcome, на случай если кто-то из участников сообщества захочет попробовать исправить это.

TurtleStack · 07.Февраль.2024 17:29:33

Я заинтересован в реализации исправления для этого. Я новичок здесь, есть ли процесс назначения для тем pr-welcome?

featheredtoast · 07.Февраль.2024 19:10:58

Привет и добро пожаловать! Комментарий «работаю над этим» вполне достаточен для процесса выполнения задания. Отправьте свой PR на GitHub, когда будете готовы, и мы его посмотрим

TurtleStack · 07.Февраль.2024 21:47:29

PR отправлен. Спасибо!

github.com/discourse/discourse

FIX: Hide timer info on topic status toggle (#25596)

main ← turtlestackdev:fix/topic_timer_visible_after_status_toggle

merged 04:40PM - 14 Feb 24 UTC

turtlestackdev

+62 -0

Fix for https://meta.discourse.org/t/minor-ui-bug-with-manually-closing-a-topic-…that-has-a-topic-timer-set-for-auto-close-in-the-future/289482 When an auto close/open timer is set and the user manually opens/closes the topic, the timer info remains displayed until the page is refreshed. The changes here fix that by adding a `@statusUpdate` property to the timer info within the topic template. The `@statusUpdate` property is read in the `topic-timer-info` component. When this value is defined, and the timer `statusType` matches the the topic status, the timer info `showTopicTimer` and `executedAt` values are set to null. The latter is modified to prevent the info from being incorrectly displayed if the user toggles the status back to open. The `@statusUpdate` property is set to the value of the topic `topic_status_update` which is undefined by default. After the topic controller `toggleClosed` function finishes, this value is updated with the return value from the API. This in turn triggers another render of the `topic-timer-info` component which now has the information it needs to hide itself. This does trigger an additional render of the timer info after the hooks are fired, but I felt it was acceptable. Three acceptance tests have been added to topic-edit-timer-test.js. - auto-close timer is hidden after manually closing the topic - auto-open timer is hidden after manually opening the topic - auto-close timer is hidden after manually closing and then re-opening the topic The topic test fixture needed a minor adjustment to enable toggling the topic status.

Тема		Ответов	Просм.
Closing a topic manually doesn't clear the topic timer's automatic closure UX topic-timers	5	794	11.12.2018
Topic timer text not removed when timer fires and topic is open UX topic-timers	2	678	13.06.2019
This topic will close a few seconds after the last reply Bug	8	686	15.02.2021
Temporary Timed Closure of topic not re-opening topic Bug	3	889	23.08.2017
Also update topic timer UI for auto-delete replies, auto-close after last post UX topic-timers	4	1093	25.09.2025

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

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