[Plugin] Notifications system (1.0)

MultiformeIngegno

  • Posts: 1,337
[Plugin] Re: Notifications system (1.0)
« Reply #75, on March 28th, 2013, 12:18 AM »
Ok phew. I feared I couldn't even explain a basic thing like that in English :P

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #76, on March 28th, 2013, 04:17 PM »
Meh forget about it, it was actually a stupid joke :P.

I've been thinking, is it a good idea to move the notification list (all of them) to the profile area instead of an independent action? The notifications action will still remain still it's used for more than that but the list goes into the profile. XF does something similar and it's not a bad idea.
The way it's meant to be

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #77, on March 28th, 2013, 04:19 PM »
That works for me.

Also, when looking through the code the other day, I realised the subtle change that occurs... it actually replaces the old notifications area for board/topic notifications by replacing area=notification in the profile. I take it that was intentional?
When we unite against a common enemy that attacks our ethos, it nurtures group solidarity. Trolls are sensational, yes, but we keep everyone honest. | Game Memorial

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #78, on March 28th, 2013, 04:31 PM »
Quote from Arantor on March 28th, 2013, 04:19 PM
That works for me.

   Also, when looking through the code the other day, I realised the subtle change that occurs... it actually replaces the old notifications area for board/topic notifications by replacing area=notification in the profile. I take it that was intentional?
Yes, the entire old notification and subscription system is meant to be replaced by this (hence the entire reason I added e-mail grouping and notifications)

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #79, on March 28th, 2013, 04:32 PM »
Just checking ;)

That would also mean we need to figure out some way of getting people to notify for a given board or topic that they're watching... and that does need to be in the core.

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #80, on March 28th, 2013, 04:33 PM »
Quote from Arantor on March 28th, 2013, 04:32 PM
Just checking ;)

   That would also mean we need to figure out some way of getting people to notify for a given board or topic that they're watching... and that does need to be in the core.
Already taken care of, just need to be incorporated into core. That part is handled by my notification subscriptions core + subscription handlers. I have coded them for topics, need to work on board but it's pretty simple.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #81, on March 28th, 2013, 04:37 PM »
Is that notification on 'new topics', 'new posts in topics you're watching' and 'new posts in topics you're watching but only for the first unread post'?

As for boards, is that 'new topics', 'new topics and new posts in topics you're watching', 'new topics and all new posts'?

There are people who want this stuff >_<

(I'm not sure whether it *should* be, I'm just trying to get a feel for what it currently is and what the plans are for it rather than whether it should do these things.)

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #82, on March 28th, 2013, 04:39 PM »
Quote from Arantor on March 28th, 2013, 04:37 PM
Is that notification on 'new topics', 'new posts in topics you're watching' and 'new posts in topics you're watching but only for the first unread post'?

   As for boards, is that 'new topics', 'new topics and new posts in topics you're watching', 'new topics and all new posts'?

   There are people who want this stuff >_<

(I'm not sure whether it *should* be, I'm just trying to get a feel for what it currently is and what the plans are for it rather than whether it should do these things.)
By default it's for new posts but can be extended in multiple ways to incorporate those option. By automatically subscribing to topics you're watching (it also has the option to automatically subscribe to topics you post or reply, I'm not sure if it works ATM :P), and by giving options. I haven't done it but porting those options should be a simple affair. What is the difference between "notify" and "watching" a topic? Seems redundant to me?

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #83, on March 28th, 2013, 04:43 PM »
Quote
What is the difference between "notify" and "watching" a topic? Seems redundant to me?
Oh, it's the same thing, just different terminology (and more sane terminology than 'Notify' seems to be). But there is a big difference between 'all new' and 'new in things you're specifically interested in'. I can well imagine some people wanting notifications on all posts, and some people only wanting notifications on posts they were watching.

(And I can imagine admins wanting to beat their users with a stick YOU WILL HAVE NOTIFICATIONS ON ALL POSTS AND YOU WILL LIKE IT >_<)

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #84, on March 28th, 2013, 04:47 PM »
Totally unrelated bug, I can't do cmd + b for bold on Mac.
Quote from Arantor on March 28th, 2013, 04:43 PM
Oh, it's the same thing, just different terminology (and more sane terminology than 'Notify' seems to be). But there is a big difference between 'all new' and 'new in things you're specifically interested in'. I can well imagine some people wanting notifications on all posts, and some people only wanting notifications on posts they were watching.
I'm going to assume you meant topics they are watching, because watching posts is not only creepy, it's useless :P. But they are options that aren't hard to implement (i.e. difference between notify board and notify topic), plus the notify on all vs notify on unread is something that's easily resolvable via handleMultiple as far as I can think.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #85, on March 28th, 2013, 04:51 PM »
Well, yes, that's what I meant :P But you see what I'm getting at - some people will want notifications on *everything* and some just on what interests them.
Quote
plus the notify on all vs notify on unread is something that's easily resolvable via handleMultiple as far as I can think.
Sounds legit :bravo:

Dragooon

  • I can code! Really!
  • polygon.com has to be one of the best sites I've seen recently.
  • Posts: 1,841
[Plugin] Re: Notifications system (1.0)
« Reply #86, on March 28th, 2013, 04:52 PM »
And giving them that option won't be hard. Hell I can easily make a plugin that allows a person to subscribe to another member. Not sure if it'll be well received :P.

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278
[Plugin] Re: Notifications system (1.0)
« Reply #87, on March 28th, 2013, 04:54 PM »
It's no more creepy than Facebook is, after all ;)

(I think XF has that as an option, actually)

MultiformeIngegno

  • Posts: 1,337

Arantor

  • As powerful as possible, as complex as necessary.
  • Posts: 14,278