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.

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

Strip the exception type when showing a checkArgument message to the user #2620

Closed
devinbileck opened this issue Apr 2, 2019 · 3 comments
Closed

Comments

@devinbileck
Copy link
Member

When a checkArgument fails and raises an exception, it is shown to the user with the exception type (java.lang.IllegalArgumentException). It may be confusing to show that, and instead should just show the message.
image

Related to #2619

@devinbileck
Copy link
Member Author

As suggested in #2614 (comment), a better implementation would be input validation where a more user friendly and translated error is shown (at text field validation).

ManfredKarrer added a commit that referenced this issue Apr 3, 2019
@ManfredKarrer
Copy link
Contributor

Excpetion class name is not displayed anymore as I use getMessage(). But still would be good to show a translated and more user friendly popup.

@ManfredKarrer
Copy link
Contributor

Close as it is covered in #2619

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants