ondrej
16 Fevereiro , 2021 09:38
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 Curtiram
sam
(Sam Saffron)
16 Fevereiro , 2021 10:33
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 Curtiram
ondrej
16 Fevereiro , 2021 10:36
4
Sim, consigo reproduzir aqui. Vou deixá-lo como está. Obrigado pelo retorno.
3 Curtiram
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 Curtiram
ondrej
18 Fevereiro , 2021 11:01
6
Oi Kris, está funcionando bem agora, obrigado!
2 Curtiram