search

NebulousLabs / Sia-UI

A Graphical Frontend for Sia - https://sia.tech
MIT License
389 stars 109 forks source link

Error messages #120

Closed lukechampine closed 9 years ago

lukechampine commented 9 years ago

Many of our error messages are too long, and don't fit in the notification window. We need some way of displaying or capturing these errors in full. They are important for debugging purposes.

DavidVorick commented 9 years ago

logging? Errors plugin?

Might not be necessary to expose them to the user, but maybe there should be some data structure that holds the whole notification history

lukechampine commented 9 years ago

I think logging to a file would be best. When a user gets an error we want to be able to point them to a specific record of the full error message, so that we can properly diagnose it.

DavidVorick commented 9 years ago

do we want to log all notifications, or only error notifications?

Mingling94 commented 9 years ago

Definitely just error notifications. There's only so many times one can read "Sia client up to date!"

DavidVorick commented 9 years ago

Maybe a sign that we don't need to display a notification about being up to date.

lukechampine commented 9 years ago

There's not much value in "Address created!" or "Wallet unlocked!" either. Unless you want to trace every action of the user, so as to replay precisely what they did. But logging just error messages seems pretty sane to me.

DavidVorick commented 9 years ago

yeah that makes sense