Imposta un timer per l’argomento per chiudere un argomento tra x giorni
Chiudi manualmente l’argomento
Puoi vedere una riga che mostra ancora che l’argomento si chiuderà tra x giorni
Fai clic sull’icona del cestino su questo e otterrai un messaggio di errore:
Ovviamente è un problema minore, amo Discourse e tutto quello che state facendo, ma ho pensato che valesse la pena menzionarlo
5 Mi Piace
sam
22 Dicembre 2023, 2:24am
2
Grazie per averlo segnalato, ho aggiunto pr-welcome
2 Mi Piace
Sono interessato a implementare una correzione per questo. Sono nuovo qui, esiste un processo di assegnazione per gli argomenti pr-welcome
3 Mi Piace
Ciao e benvenuto! Il tuo commento “ci sto lavorando” è più che sufficiente per il processo di assegnazione: inviaci la tua PR su GitHub quando sarà pronta e potremo darle un’occhiata
3 Mi Piace
La PR è stata inviata. Grazie!
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 Mi Piace
Grazie