Review Request 109557: Do not show Tp error messages to the user

[Martin Klapetek]

> On March 18, 2013, 9:32 a.m., Martin Klapetek wrote:
> > While in principle I agree, how do you inform the user that the operation he tried to do has failed (and nothing is happening and user has no clue why)?
> 
> David Edmundson wrote:
>     Is that any better than "org.telepathy.freedesktop/Gabble/david%20gmail%23.com/ladsjfasr Capability not available"?
>     
>     That doesn't tell me they were unable to move a contact into a different group because Facebook doesn't support it.
>     
>     They can see it hasn't worked because nothing happens.
>     
>     If we want to report back errors we shouldn't be using genericOperationFailed.

> Is that any better than "org.telepathy.freedesktop/Gabble/david%20gmail%23.com/ladsjfasr Capability not available"?

Yes, it's better than nothing happening. This you can google or ask someone about and gives you /some/ feedback, when compared to nothing happening. The user will try again, nothing happens again. That imho creates a sense of broken software. 

So it's better to have a broken notification text than broken software.


- Martin


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://git.reviewboard.kde.org/r/109557/#review29424
-----------------------------------------------------------


On March 18, 2013, 2:28 a.m., David Edmundson wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> http://git.reviewboard.kde.org/r/109557/
> -----------------------------------------------------------
> 
> (Updated March 18, 2013, 2:28 a.m.)
> 
> 
> Review request for Telepathy.
> 
> 
> Description
> -------
> 
> Do not show Tp error messages to the user
> 
> 
> Diffs
> -----
> 
>   main-widget.cpp 90093be18d27dd672628624a0788b9c2ceda4147 
> 
> Diff: http://git.reviewboard.kde.org/r/109557/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> David Edmundson
> 
>

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/kde-telepathy/attachments/20130318/9ab3b554/attachment.html>