ondrej
Fevereiro 16, 2021, 9:38am
1
Olá,
O problema é que, se você acessar /new e clicar em ‘Ignorar Novo’, a página é recarregada, mas os tópicos não desaparecem.
2 curtidas
sam
(Sam Saffron)
Fevereiro 16, 2021, 10:33am
3
Temos mexido na implementação para adicionar suporte ao ‘dismiss new’ em tags. Suponha que você tenha uma reprodução no meta? Pode deixar como está e nós vamos depurar.
3 curtidas
ondrej
Fevereiro 16, 2021, 10:36am
4
Sim, consigo reproduzir aqui. Vou deixá-lo como está. Obrigado pelo retorno.
3 curtidas
Obrigado por relatar esse bug. Eu o corrigi com este PR. Posso pedir que você o teste novamente?
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 curtidas
ondrej
Fevereiro 18, 2021, 11:01am
6
Oi Kris, está funcionando bem agora, obrigado!
2 curtidas