Assistir a uma categoria não causa notificações push

Quando tenho uma categoria definida como “assistir primeiro post” (“Você será notificado do primeiro post em cada novo tópico nessas categorias.”), recebo notificações push no meu telefone quando um tópico é criado em uma categoria, e também recebo notificações push para respostas subsequentes.

Mas quando tenho a categoria definida como “assistir” (“Você será notificado de todos os novos posts e tópicos.”), não recebo notificações push.

Verificando o código e encontro uma condição NOTIFIABLE_TYPES.include?(type)

NOTIFIABLE_TYPES inclui watching_first_post e posted, mas não inclui watching_category_or_tag.

Isso é intencional? Sempre pensei que assistir fosse um superconjunto de assistir primeiro post?

(Semelhante/relacionado: Watching First Post does not send push notifications e esta alteração)

EDIT Estou atualmente executando alguns fóruns que estou usando com uma versão modificada que adiciona watching_category_or_tag e com push notification time window mins definido como 0, minha experiência de usuário passou de ‘meh’ para ‘ótima’. Então, se isso não for um bug, considere-o uma solicitação de recurso :wink:

8 curtidas

Eu mesclei um commit que adiciona watching_category_or_tag a NOTIFIABLE_TYPES exatamente como você descreveu! Obrigado pelo relatório de bug e pela solução :chefs_kiss:

7 curtidas

Obrigado @markvanlan!!

… Acabei de descobrir que isso também requer uma string de localidade en.discourse_push_notifications.popup.watching_category_or_tag

1 curtida

Ótima observação, claro que o tipo está vinculado a uma tradução! Mas… estou curioso sobre o melhor texto. O de watching_first_post está abaixo. Eu me pergunto se a tradução deveria ser X criou uma nova postagem, em vez de X criou um novo tópico, já que pode ser um novo tópico, mas mais provavelmente é uma resposta.

Para deixar a string mais elaborada, teremos que modificar PushNotificationPusher. Por enquanto, criarei um PR para implementar minha sugestão e poderemos fazer um aprimoramento depois.

watching_first_post: '%{username} criou um novo tópico "%{topic}" - %{site_title}'
watching_category_or_tag: '%{username} criou uma nova postagem "%{topic}" - %{site_title}'

EDIT: Adicionei um commit com a tradução. Vou acompanhar amanhã e ver sobre proporcionar uma experiência mais rica que linka diretamente para a postagem, com uma nota sobre qual categoria ou tag está associada a ela. FIX: Add translation for new push notification (#24203) · discourse/discourse@1d96b0a · GitHub

4 curtidas

Acabei de enviar outro commit que melhora um pouco o texto da notificação push. Da descrição do PR:

Agora, para tópicos novos, ele exibirá:

username criou um novo tópico "título aqui!" - Discourse do Mark

E para posts novos:

username postou em "título aqui!" - Discourse do Mark


Depois de investigar o código, seria uma grande mudança fazer o texto ler algo como: “X postou um novo tópico na categoria/tag Y”. Este é um bom passo na direção certa e podemos seguir com uma mudança muito mais complexa se for necessária no futuro.

Obrigado novamente @RGJ!

3 curtidas

Este tópico foi fechado automaticamente após 13 horas. Novas respostas não são mais permitidas.