Upvote notification tweaks

[boomzilla]

Back home we've had complaints about the number of notifications people receive due to upvotes. We are a community that likes to upvote and it is a lot of notifications.

So I've been working on changing the way upvote notifications are handled. Here's the drop down from the user settings page:

The code is up on github if anyone is interested in testing or just taking a look:
NodeBB - upvote-notifications branch
Persona Theme - upvote-notifications branch
Vanilla Theme - upvote-notifications branch

I hope to turn those into pull requests soon.

[PitaJ]

@boomzilla Very cool. I'd like it if reddit were to give me notifications when my posts receive landmarks of upvotes.

Maybe it should be an exponential growth kind of thing? So like if the log-base-10 of the # of upvotes is a whole number, it would give you a notification?

Here's some code:

const isWhole = num => num % 1 === 0;

const shouldNotify = upvotes => {
  // notify if [a power of 10] or if [5 * (a power of 10)]
  if (isWhole(Math.log10(upvotes)) || isWhole(Math.log10(upvotes / 5))) {
    return true;
  }
  return false;
}

// test
Array.from({ length: 20 }, (x, i) => {
  const p = Math.pow(10, Math.floor(i / 4));
  if (i % 4 === 0) {
    return p;
  }
  if (i % 4 === 3) {
    return p * 7;
  }
  if (i % 4 === 2) {
    return p * 5;
  }
  return p * 3;
}).map(shouldNotify);
// should be an array with alternating true/false

[boomzilla]

Yeah, we discussed stuff like that. I suspect that the "every 50 votes" is probably not going to be very common as it is. Unless you really are running the next reddit on nodebb.

[PitaJ]

@boomzilla It's probably still best to go with the most future-proof algorithm while it hasn't been established yet, just in case.

[abarker]

@PitaJ That actually looks pretty slick. If I'm reading that right, your code should notify on upvotes 1, 5, 10, 50, 100, 500, …

That seems like a pretty good fit for the middle-ground option.

[PitaJ]

@abarker yeah, that's exactly what it should do. Haven't tested it yet. One reservation I have is that it may be more representative to use 3 instead of 5 since, on a log scale, 3 is closer to a half-decade. That may not be seen as true by actual users, though.

[boomzilla]

Submitted a pull request:
https://github.com/NodeBB/NodeBB/pull/4591

boomzillawtf created this issue in NodeBB/NodeBB

closed Upvote notifications #4591