Définir un minuteur de sujet pour fermer un sujet dans x jours
Fermer le sujet manuellement
Vous pouvez voir une ligne indiquant toujours que le sujet se fermera dans x jours
Cliquez sur l’icône de corbeille sur celle-ci et vous obtenez un message d’erreur :
Évidemment, c’est un problème mineur, j’adore Discourse et tout ce que vous faites, mais j’ai pensé que cela valait la peine d’être mentionné :smile : Merci.
5 « J'aime »
sam
Décembre 22, 2023, 2:24
2
Merci de l’avoir soulevé, j’ai mis pr-welcome
2 « J'aime »
Je suis intéressé par la mise en œuvre d’une solution pour cela. Je suis nouveau ici, existe-t-il un processus d’attribution pour les sujets pr-welcome
3 « J'aime »
Bonjour et bienvenue ! Votre commentaire disant « en cours » suffit amplement pour un processus d’affectation - Envoyez votre PR sur GitHub quand il sera prêt, et nous pourrons y jeter un œil
3 « J'aime »
La PR a été envoyée. Merci !
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 « J'aime »
