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

[David Edmundson]

> 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)?

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.


- David


-----------------------------------------------------------
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/7f989f76/attachment.html>