show error string instead error number
ClosedPublic

Authored by mkoller on Dec 8 2017, 12:51 PM.

Details

Summary

Instead of just showing an error number when notifying the user, show an error string, since
"Failed to transport message. 3" does not really say much to the user.

This is possible now with the pending patch https://phabricator.kde.org/D9249

Now at least I get "Failed to transport message. Host not found"

Diff Detail

Repository
R733 KSmtp
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
mkoller requested review of this revision.Dec 8 2017, 12:51 PM
mkoller created this revision.

Since https://phabricator.kde.org/D9249 was already accepted a week ago, can I commit this now ?

dvratil accepted this revision.Dec 18 2017, 10:02 AM

Thanks, this was the original and intended behavior. I must've accidentally committed the error code while I was debugging something.

This revision is now accepted and ready to land.Dec 18 2017, 10:02 AM
This revision was automatically updated to reflect the committed changes.