Lorsque j’ai une catégorie définie sur « regarder le premier message » (« Vous serez informé du premier message de chaque nouveau sujet dans ces catégories. »), je reçois des notifications push sur mon téléphone lorsqu’un sujet est créé dans une catégorie, et je reçois également des notifications push pour les réponses ultérieures.
Mais lorsque j’ai la catégorie définie sur « regarder » (« Vous serez informé de tous les nouveaux messages et sujets »), je ne reçois pas de notifications push.
En vérifiant le code, je trouve une condition NOTIFIABLE_TYPES.include?(type)
NOTIFIABLE_TYPESinclutwatching_first_post et posted, mais il n’inclut pas watching_category_or_tag.
Est-ce intentionnel ? J’ai toujours pensé que regarder était un sur-ensemble de regarder le premier message ?
EDIT Je gère actuellement quelques forums que j’utilise moi-même avec une version patchée qui ajoute watching_category_or_tag et avec push notification time window mins défini sur 0, mon expérience utilisateur est passée de « bof » à « super ». Donc, si ce n’est pas un bug, considérez-le comme une demande de fonctionnalité
Ah, bonne remarque, bien sûr que le type est lié à une traduction ! Mais… je suis curieux de savoir quel est le meilleur texte. Celui pour watching_first_post est ci-dessous. Je me demande si la traduction devrait être X a créé un nouveau post, plutôt que X a créé un nouveau sujet, car il pourrait s’agir d’un nouveau sujet mais plus probablement d’une réponse.
Pour rendre la chaîne plus sophistiquée, nous devrons modifier PushNotificationPusher. Pour l’instant, je vais créer une PR pour implémenter ma suggestion et nous pourrons y donner suite avec une amélioration.
watching_first_post: '%{username} a créé un nouveau sujet « %{topic} » - %{site_title}'
watching_category_or_tag: '%{username} a créé un nouveau post « %{topic} » - %{site_title}'
J’ai juste poussé un autre commit qui améliore un peu le texte des notifications push. D’après la description de la PR :
Maintenant, pour les nouveaux sujets, il sera lu :
username a créé un nouveau sujet « titre ici ! » - Discourse de Mark
Et pour les nouveaux messages :
username a posté dans « titre ici ! » - Discourse de Mark
Après avoir examiné le code, ce serait un changement énorme d’avoir le texte lu comme ceci : « X a posté un nouveau sujet dans la catégorie/le tag Y ». C’est un bon pas dans la bonne direction et nous pourrons faire un changement beaucoup plus complexe à l’avenir si nécessaire.
