Legen Sie einen Themen-Timer fest, um ein Thema in x Tagen zu schließen
Schließen Sie das Thema manuell
Sie können immer noch eine Zeile sehen, die anzeigt, dass das Thema in x Tagen geschlossen wird
Klicken Sie auf das Papierkorbsymbol hier und Sie erhalten eine Fehlermeldung:
Offensichtlich sehr geringfügig, ich liebe Discourse und alles, was Sie tun, aber ich dachte, es wäre erwähnenswert
5 „Gefällt mir“
sam
22. Dezember 2023 um 02:24
2
Danke, dass Sie es angesprochen haben. Ich habe hierfür pr-welcome
2 „Gefällt mir“
Ich bin daran interessiert, eine Korrektur dafür zu implementieren. Ich bin neu hier, gibt es einen Zuweisungsprozess für #pr-welcome-Themen ?
3 „Gefällt mir“
Hallo und willkommen! Ihr Kommentar „working on it“ genügt für den Zuweisungsprozess – senden Sie Ihren PR an GitHub, wenn er fertig ist, und wir können ihn uns ansehen
3 „Gefällt mir“
Der PR wurde gesendet. Danke!
main ← turtlestackdev:fix/topic_timer_visible_after_status_toggle
opened 09:46PM - 07 Feb 24 UTC
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.
5 „Gefällt mir“
Danke