[Plugin] Notifications system (1.0)

Nao

  • Dadman with a boy
  • Posts: 16,079
[Plugin] Re: Notifications system (1.0)
« Reply #225, on June 28th, 2013, 09:15 PM »
Notification code is up and fixed, I also fixed the Move thing, which had a broken preview, even in the SVN version.

Oh, and btw... Why is the Move notifier called 'move', when others have a 's' at the end..? (likes, mentions)
We should standardize these, too...

And also strtolower the entire class names, because, well, I'm definitely not into these...

live627

  • Should five per cent appear too small / Be thankful I don't take it all / 'Cause I'm the taxman, yeah I'm the taxman
  • Posts: 1,670
[Plugin] Re: Notifications system (1.0)
« Reply #226, on June 29th, 2013, 01:42 AM »
Quote
Oh, and btw... Why is the Move notifier called 'move', when others have a 's' at the end..? (likes, mentions)
I  named  it but  you can change it. My reasoning was that it always notifies on a single topic because one topic was moved at a time.
A confident man keeps quiet.whereas a frightened man keeps talking, hiding his fear.

Nao

  • Dadman with a boy
  • Posts: 16,079
[Plugin] Re: Notifications system (1.0)
« Reply #227, on June 29th, 2013, 11:47 AM »
I'm more suspicious of adding an 's' than removing it, actually... ;)
"It's a move", "it's a like", etc...