Seguire una categoria non causa notifiche push

[RGJ (Richard - Communiteq)]

Quindi, quando imposto una categoria su “watch first post” (“Riceverai una notifica per il primo post di ogni nuovo argomento in queste categorie.”), ricevo notifiche push sul mio telefono quando viene creato un argomento in una categoria e ricevo notifiche push anche per le risposte successive.

Ma quando imposto la categoria su “watching” (“Riceverai una notifica per tutti i nuovi post e argomenti”), non ricevo notifiche push.

Controllando il codice trovo una condizione NOTIFIABLE_TYPES.include?(type)

NOTIFIABLE_TYPES include watching_first_post e posted, ma non include watching_category_or_tag.

È intenzionale? Ho sempre pensato che watching fosse un superset di watching first post?

(Simile/correlato: Watching First Post does not send push notifications e questo cambiamento)

EDIT Attualmente gestisco alcuni forum che uso personalmente con una versione patchata che aggiunge watching_category_or_tag e con push notification time window mins impostato a 0 la mia esperienza utente è passata da “meh” a “ottima”. Quindi, se questo non è un bug, consideralo una richiesta di funzionalità

[markvanlan]

Ho unito un commit che aggiunge watching_category_or_tag a NOTIFIABLE_TYPES proprio come hai delineato! Grazie per il bug report e la soluzione

[RGJ (Richard - Communiteq)]

Grazie @markvanlan!!

… Ho appena scoperto che questo richiede anche una stringa di localizzazione en.discourse_push_notifications.popup.watching_category_or_tag

[markvanlan]

Ottima osservazione, certo che il tipo è legato a una traduzione! Ma… sono curioso riguardo al testo migliore. Quello per watching_first_post è qui sotto. Mi chiedo se la traduzione dovrebbe essere X ha creato un nuovo post, piuttosto che X ha creato un nuovo argomento, poiché potrebbe essere un nuovo argomento ma più probabilmente è una risposta.

Per rendere la stringa più elaborata dovremo modificare PushNotificationPusher. Per ora creerò una PR per implementare il mio suggerimento e potremo seguirla con un miglioramento.

watching_first_post: '%{username} ha creato un nuovo argomento "%{topic}" - %{site_title}'
watching_category_or_tag: '%{username} ha creato un nuovo post "%{topic}" - %{site_title}'

EDIT: Aggiunto un commit con la traduzione. Domani seguirò per vedere come offrire un’esperienza più ricca che rimandi direttamente al post, con una nota su quale categoria o tag è associato. FIX: Add translation for new push notification (#24203) · discourse/discourse@1d96b0a · GitHub

[markvanlan]

Ho appena inviato un altro commit che migliora leggermente il testo delle notifiche push. Dalla descrizione della PR:

Ora per i nuovi topic leggerà:

username ha creato un nuovo topic \"titolo qui!\" - Discourse di Mark

E per i nuovi post:

username ha postato in \"titolo qui!\" - Discourse di Mark

---

Dopo aver analizzato il codice, sarebbe un cambiamento enorme far leggere il testo come: “X ha postato un nuovo topic nella categoria/tag Y”. Questo è un buon passo nella giusta direzione e possiamo seguire con un cambiamento molto più complesso se sarà richiesto in futuro.

Grazie ancora @RGJ!