ondrej
16 Febbraio 2021, 9:38am
1
Ciao,
Il problema è che se vai a /new e premi “Nascondi nuovi”, la pagina si ricarica ma i topic non scompaiono.
2 Mi Piace
sam
(Sam Saffron)
16 Febbraio 2021, 10:33am
3
Abbiamo lavorato sull’implementazione per aggiungere il supporto a ‘dismiss new’ sui tag; supponiamo che tu abbia una riproducibilità su meta? Puoi lasciarlo così com’è e noi faremo il debug.
3 Mi Piace
ondrej
16 Febbraio 2021, 10:36am
4
Sì, riesco a riprodurlo qui. Lo lascio così com’è. Grazie per avermi risposto.
3 Mi Piace
Grazie per aver segnalato quel bug. L’ho corretto con questa PR. Posso chiederti di riprovare a testarlo?
master ← KrisKotlarek:improvement-for-dismiss-topic
merged 11:39PM - 17 Feb 21 UTC
The bug was mentioned on meta: https://meta.discourse.org/t/pressing-dismiss-new… -doesnt-clear-new-topics/179858
Problem is that sometimes the user has TopicUser records with `last_read_post_number` set as NULL. In that case, the topic is still "new" to them and should be dismissed when they click dismiss button.
In addition, I added that condition to post_migration and bumped the number to fix existing records. Migration is written to be idempotent so it will make no harm to already deployed instances.
2 Mi Piace
ondrej
18 Febbraio 2021, 11:01am
6
Ciao Kris, ora funziona perfettamente, grazie!
2 Mi Piace