6.0.0-beta1
7/10/25

[#2332] Provide a notification that a task has been added to your tasklist
Summary Provide a notification that a task has been added to your tasklist
Queue Nag
Queue Version HEAD
Type Enhancement
State Resolved
Priority 1. Low
Owners
Requester kevin_myer (at) iu13 (dot) org
Created 07/25/2005 (7290 days ago)
Due
Updated 11/25/2005 (7167 days ago)
Assigned
Resolved 11/25/2005 (7167 days ago)
Milestone
Patch No

History
11/25/2005 03:58:02 PM Jan Schneider Comment #10
State ⇒ Resolved
Reply to this comment
Slightly tweaked and committed, thanks. Also merged to FRAMEWORK_3 to 
line up Nag 2.1 with Kronolith 2.1.
11/25/2005 02:51:51 PM kevin_myer (at) iu13 (dot) org Comment #9
New Attachment: nag-notification[2].patch Download
Reply to this comment
Looks good. The only nit I found is that you probably want to check
if the return value of _add() is an error and don't proceed in this
case.
Are you going to update the patch to hook into editing and deleting,
or should I commit it for now?
I at least thought about doing the check for the return value but I 
guess I never added it.  Here's an updated version and it handles 
editing and deleting too.


11/24/2005 10:49:48 AM Jan Schneider Comment #8 Reply to this comment
Looks good. The only nit I found is that you probably want to check if 
the return value of _add() is an error and don't proceed in this case.

Are you going to update the patch to hook into editing and deleting, 
or should I commit it for now?
11/23/2005 10:03:03 PM kevin_myer (at) iu13 (dot) org Comment #7 Reply to this comment
Also, I left $priority in as a parameter for notification.  I was 
thinking that it could be used to set the X-Priority header in the 
email that is generated.
11/23/2005 10:00:23 PM kevin_myer (at) iu13 (dot) org Comment #6
New Attachment: nag-notification[1].patch Download
Reply to this comment
Here is a patch that implements notifications as suggested.  It 
currently only handles notifications when tasks are added but could be 
extended in the same manner for modifications or deletions.  I added a 
$uid parameter to the kolab driver, only because it existed in the SQL 
driver, but it looks like they are autogenerated anyway, so it might 
not be necessary.  The notification could be changed to work with a 
user's 'date_format' and 'time_format' preferences, but I think that 
would only make sense if time_format was in the horde preference scope 
and not imp.
11/13/2005 11:20:32 AM Jan Schneider Comment #5
State ⇒ Feedback
Reply to this comment
How about splitting up add() to Driver::add() and Driver_foo::_add()?
History logging and notificiation would be done in add() and the
actual adding in _add().
If I understand you right, that would mean leaving the backend
specific code in Driver_foo::add() and drawing the common code into
Driver::add()?
Not quite, the backend specific code goes into Driver_foo::_add(), 
because you will call these methods from the parent class, not the 
parent class' method from the inherited class.
I really only wanted to get notifications working, but if it makes
sense to make those changes, I can certainly try :)
It makes a lot of sense, because you would make notifications work 
with *all* driver at once.
11/13/2005 01:58:51 AM kevin_myer (at) iu13 (dot) org Comment #4 Reply to this comment
How about splitting up add() to Driver::add() and Driver_foo::_add()?
History logging and notificiation would be done in add() and the
actual adding in _add().
If I understand you right, that would mean leaving the backend 
specific code in Driver_foo::add() and drawing the common code into 
Driver::add()?



I really only wanted to get notifications working, but if it makes 
sense to make those changes, I can certainly try :)
11/03/2005 11:15:55 AM Jan Schneider Comment #3 Reply to this comment
How about splitting up add() to Driver::add() and Driver_foo::_add()? 
History logging and notificiation would be done in add() and the 
actual adding in _add().
11/02/2005 11:24:41 PM kevin_myer (at) iu13 (dot) org Comment #2
New Attachment: nag-notification.patch Download
Reply to this comment
Here's an attempt to add email notifications for tasks.  I found it 
simpler to model the Kronolith code for this, so this also implements 
a (largely unused) Nag_Task object, based on the Kronolith_Event 
object.  Notifications are currently only supported in the SQL driver, 
although it should be easily adapted to the Kolab driver, but I have 
no way of testing that.
07/25/2005 03:22:04 PM Jan Schneider State ⇒ Accepted
 
07/25/2005 01:58:20 PM kevin_myer (at) iu13 (dot) org Comment #1
Priority ⇒ 1. Low
Type ⇒ Enhancement
Summary ⇒ Provide a notification that a task has been added to your tasklist
Queue ⇒ Nag
State ⇒ New
Reply to this comment
This feature would be similar to the option available in Kronolith, 
where you can choose the types of actions for an event on your 
calendar that generate a notification for you.  Example, if someone 
adds a task to your tasklist, you could choose to be notified by email 
that a new task has been addded.

Saved Queries