Status notification should occur after OmniHandler execution about omnidroid HOT 19 CLOSED

This should be part of a "Debug" mode option that users can enable/disable on 
the
"Settings" page.  Instead of a "notification" we may prefer a "toast" when this 
debug
option is enabled.

so now there is "toast" in GMailService and SMSService. So should we change it 
all to notifications? or just add toast? when smsservice is triggered it shows 
some message(something like "smsservice started"), but the final message 
whether message was sent or not could be notification.

Sorry, I didn't really think this through completely when I posted it.

I think by default a toast should occur for each rule that is applied.  Then if 
an action deems it appropriate to send it's own toast such as "SMS Sent 
Successfully" that fall on the Action object to perform that.  This issue is 
about implementing the former.

In addition I think it would be of benefit if we could have a "Send 
Notification on Rule" option in the Settings activity that instead of using a 
Toast it sends a Notification about that Rule being applied.

I think rules should default to Toasts to avoid unwanted notification clutter, 
but users should be able to toggle it in the event that they want to easily see 
that rules are being applied without having to sift through the logs or 
actively monitoring their screen.

Thoughts?

yeah we could have notification on/off in context menu(along with delete, 
delete all, enable/disable all, edit) in saved rules activity. but still think 
it should be enabled by default. you don't want users not being aware of that 
their phone is on silent and blaming omnidroid for missed calls. or we could 
have dialog box after you save a rule.

Defaulting to on for notifications is fine with me as long as we have a way to 
turn it off.

Just to clarify, the options that apply to all rules (enable all, disable all, 
delete all, toggle notify on/off) should be on the options menu, while the item 
specific options (delete, edit, view, etc.) should be on context menus.

I don't like idea of having both. 

You don't like having both an OptionsMenu and a ContextMenu or a both 
Notifications and Toasts?

the menus. but this should be discussed somewhere else, we should open new 
issue about improving saved rules activity.

Agreed.  For reference, Issue111 and Issue66 are about Saved Rules Activity UI 
changes.

http://codereview.appspot.com/1726050/show

committed r807

I just realized that this change looks like it only affects OmniActionServices. 
 I would think that the original desire would have been to have this applied 
during HandlerService processing or ActionExecutor instead of in the specific 
Action code.  That would allow these notification on all actions as opposed to 
limited to implementation by OmniActionServices.

why only OmniActionServices?
but you're right, I will consider moving that code to ResultProcessor after i'm 
done with it, and also using UtilUI showNotification, but right now we couldn't 
have it in ActionExecuter because notification message depends on whether 
actions was successfully executed or not.

Oh I see yes you're right it is included in the other actions as well.  I was 
just looking at that specific case.  But yeah, it should be moved into a more 
general location so 3rd party actions will be affected as well.  The 
ResultProcessor... that's part Issue92 that you're working on?  That seems like 
the perfect place for it.  Looking forward to it!  I'll re-mark this as Started 
and add blocked on 92.  Thanks!

I'm thinking notifications about failure shouldn't be sent until action is 
removed from failedactions table. otherwise we would get several notifications 
for one action. 

thoughts?

I agree

r839