Benachrichtigung, wenn mehrere Benutzer einen einzelnen Beitrag liken

esmon · 23. April 2020 um 16:52

Ich habe etwas im Code für Benachrichtigungen herumgesucht und festgestellt, dass diese Bedingung in widgets/notification-item.js.es6 nie erfüllt wird. Ich verwende Version 2.3.8.

Ein console.log des data-Objekts zeigt, dass data.count normalerweise undefined ist, es sei denn, es handelt sich um eine konsolidierte Benachrichtigung, bei der ein Benutzer mehr als einen deiner Beiträge liked. Außerdem ist username2 im data-Objekt nicht definiert.

Wird dieser Benachrichtigungstyp abgeschafft?

codinghorror · 11. Mai 2020 um 03:40

Ist das toter Code, @eviltrout?

sam · 11. Mai 2020 um 04:55

Ich glaube, das passiert, wenn du Benachrichtigungen für jeden Like aktiviert hast. (Ich verwende das bei Meta)

eviltrout · 11. Mai 2020 um 15:23

Unter bestimmten Umständen senden wir username2 definitiv aus:

github.com/discourse/discourse

app/services/post_action_notifier.rb

main


      
          def self.refresh_like_notification(post, read)
            return unless post && post.user_id && post.topic
          
            usernames =
              post
                .post_actions
                .where(post_action_type_id: PostActionType.types[:like])
                .joins(:user)
                .order("post_actions.created_at desc")
                .where("post_actions.created_at > ?", 1.day.ago)
                .pluck(:username)
          
            if usernames.length > 0
              data = {
                topic_title: post.topic.title,
                username: usernames[0],
                display_username: usernames[0],
                username2: usernames[1],
                count: usernames.length,
              }
              Notification.create(

esmon · 11. Mai 2020 um 16:45

Danke für die Antwort @codinghorror @eviltrout. Mir ist aufgefallen, dass dies scheinbar refaktoriert wurde und in Version 2.4.2 nun anders ist.

codinghorror · 11. Mai 2020 um 17:55

Verschiebe das in Development discussion.

Thema		Antworten	Aufrufe
Notification api data issue Support rest-api , notifications	5	96	31. Oktober 2025
More notifications for heavily-liked posts Feature	4	1372	5. Oktober 2014
Users not getting notification for every Like on replies Feature	7	1765	4. Dezember 2025
Only notified for first like Feature	9	1720	2. März 2016
A user changing their like to a reaction can lead to wrong numbers in notification menu Bug reactions , notifications	0	33	22. Februar 2026

Benachrichtigung, wenn mehrere Benutzer einen einzelnen Beitrag liken

Verwandte Themen