Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Consider giving more descriptive error message #52

Open
KSAlpha opened this issue Jan 4, 2019 · 1 comment
Open

Consider giving more descriptive error message #52

KSAlpha opened this issue Jan 4, 2019 · 1 comment
Labels
ux/ui Related to user experiences and interactions (e.g. visual representations).
Milestone
0.1.0

Comments

@KSAlpha
Copy link
Member

KSAlpha commented Jan 4, 2019

Currently Soma implementations simply passes Error to handle errors (without panicking). However these errors may mislead users when troubleshooting. We can give more descriptive messages on Error by mapping Error to SomaError or using other implementations.
@KSAlpha KSAlpha added enhancement Improvements on performance or other factors deferred Will be handled later labels Jan 4, 2019
@Qwaz
Copy link
Member

Qwaz commented Jan 14, 2019

Good article: http://blog.ploeh.dk/2014/12/23/exception-messages-are-for-programmers/

@KSAlpha KSAlpha added ux/ui Related to user experiences and interactions (e.g. visual representations). and removed enhancement Improvements on performance or other factors labels Jan 23, 2019
@KSAlpha KSAlpha removed the deferred Will be handled later label Jan 30, 2019
@KSAlpha KSAlpha added this to the 0.1.0 milestone Jan 30, 2019
@KSAlpha KSAlpha mentioned this issue Jan 30, 2019
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
ux/ui Related to user experiences and interactions (e.g. visual representations).
Projects
None yet
Milestone
0.1.0
Development

No branches or pull requests
2 participants
@Qwaz @KSAlpha