Wenn eine Kategorie auf „Ersten Beitrag beobachten“ („Sie werden über den ersten Beitrag in jedem neuen Thema in diesen Kategorien benachrichtigt“) eingestellt ist, erhalte ich Push-Benachrichtigungen auf meinem Handy, wenn ein Thema in einer Kategorie erstellt wird, und ich erhalte auch Push-Benachrichtigungen für nachfolgende Antworten.
Wenn die Kategorie jedoch auf „Beobachten“ („Sie werden über alle neuen Beiträge und Themen benachrichtigt“) eingestellt ist, erhalte ich keine Push-Benachrichtigungen.
Ich überprüfe den Code und finde eine Bedingung NOTIFIABLE_TYPES.include?(type)
NOTIFIABLE_TYPES enthält dazuwatching_first_post und posted, aber nicht watching_category_or_tag.
Ist das beabsichtigt? Ich dachte immer, „Beobachten“ sei eine Obermenge von „Ersten Beitrag beobachten“?
EDIT Ich betreibe derzeit einige Foren, die ich selbst nutze, mit einer gepatchten Version, die watching_category_or_tag hinzufügt, und mit push notification time window mins auf 0 gesetzt, ging meine Benutzererfahrung von „meh“ zu „großartig“. Wenn dies also kein Fehler ist, betrachten Sie es als Funktionswunsch
Ich habe einen Commit zusammengeführt, der watching_category_or_tag zu NOTIFIABLE_TYPES hinzufügt, genau wie Sie es beschrieben haben! Vielen Dank für den Fehlerbericht und die Lösung
… Ich habe gerade herausgefunden, dass dies auch eine Gebietsschema-Zeichenfolge erfordert en.discourse_push_notifications.popup.watching_category_or_tag
Oh, guter Fang, natürlich ist der Typ an eine Übersetzung gebunden! Aber… ich bin neugierig auf den besten Text. Der für watching_first_post ist unten. Ich frage mich, ob die Übersetzung X hat einen neuen Beitrag erstellt lauten sollte, anstatt X hat ein neues Thema erstellt, da es sich um ein neues Thema handeln könnte, aber wahrscheinlicher ist eine Antwort.
Um den String ausgefallener zu gestalten, müssen wir PushNotificationPusher ändern. Vorerst werde ich einen PR erstellen, um meinen Vorschlag umzusetzen, und wir können uns später um eine Verbesserung kümmern.
watching_first_post: '%{username} hat ein neues Thema erstellt: "%{topic}" - %{site_title}'
watching_category_or_tag: '%{username} hat einen neuen Beitrag erstellt: "%{topic}" - %{site_title}'
Ich habe gerade einen weiteren Commit gepusht, der den Text der Push-Benachrichtigungen etwas verbessert. Aus der PR-Beschreibung:
Für neue Themen wird es jetzt lauten:
username hat ein neues Thema erstellt „Titel hier!“ - Marks' Discourse
Und für neue Beiträge:
username hat in „Titel hier!“ gepostet - Marks' Discourse
Nachdem ich mir den Code angesehen habe, wäre es eine riesige Änderung, wenn der Text etwa lauten würde: „X hat ein neues Thema in der Kategorie/dem Tag Y erstellt“. Dies ist ein guter Schritt in die richtige Richtung, und wir können bei Bedarf in Zukunft eine viel komplexere Änderung vornehmen.
