Notification de plusieurs utilisateurs ayant liké un seul post

esmon · Avril 23, 2020, 4:52

J’ai fait un peu d’exploration dans le code des notifications et j’ai remarqué que cette condition n’est jamais remplie dans widgets<U+2069>/notification-item.js.es6. J’utilise la version 2.3.8.

Un console.log de l’objet data montre que data.count est généralement undefined, sauf s’il s’agit d’une notification consolidée pour laquelle un utilisateur a aimé plus d’un de vos messages. De plus, username2 n’est pas défini dans l’objet data.

Ce type de notification est-il en cours de suppression ?

codinghorror · Mai 11, 2020, 3:40

Est-ce du code mort, @eviltrout ?

sam · Mai 11, 2020, 4:55

Je pense que ce genre de chose se produit si vous avez activé la notification pour chaque like. (Je l’utilise sur Meta)

eviltrout · Mai 11, 2020, 3:23

Nous envoyons bel et bien username2 dans certaines circonstances :

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 · Mai 11, 2020, 4:45

Merci de votre réponse @codinghorror @eviltrout. J’ai réalisé que cela semble avoir été refactorisé et est différent dans la version 2.4.2 maintenant.

codinghorror · Mai 11, 2020, 5:55

Déplacement vers la discussion Development.

Sujet		Réponses	Vues
Notification api data issue Support rest-api , notifications	5	96	Octobre 31, 2025
More notifications for heavily-liked posts Feature	4	1372	Octobre 5, 2014
Users not getting notification for every Like on replies Feature	7	1765	Décembre 4, 2025
Only notified for first like Feature	9	1720	Mars 2, 2016
A user changing their like to a reaction can lead to wrong numbers in notification menu Bug reactions , notifications	0	33	Février 22, 2026

Notification de plusieurs utilisateurs ayant liké un seul post

Sujets connexes