More notifications for heavily-liked posts

(Kane York) #1

Right now, a notification is only sent for the first like on any of your posts.

But I think there’s a big difference between a post being liked once and 10 times. The second shows enjoyment of the post by many people, instead of just one.

I’m proposing that we show a notification (that gets collapsed into the previous ones, just like the n replies) for 1, 5, 10, 50, 100, 500, … likes.

Sample code for checking the number: (ruby-ish pseudocode, I don’t feel like looking up the math function names)

def should_notify?(total_likes)
  return true if total_likes == 1
  normalized = total_likes / pow(10, floor(log10(total_likes)))
  (normalized.is_integer? && (normalized == 1 || normalized == 5))

Notifications disappear on Restart / Upgrade?
Initial Discourse badge design spec
(Jeff Atwood) #2

Yes, excellent idea. The current like rollups are not optimal.

(Régis Hanol) #4

This has always bugged me. PR to fix that more than welcome! :slight_smile:

(Sam Saffron) #5

I don’t think we need this any more.

The badges fill this role, you get a badge at 10-25-50 likes which is plenty.

(Sam Saffron) #6