ModelInference / perfume-frontend

Web-based frontend for Perfume tool
Other
1 stars 0 forks source link

Pressing the "Parse Log" button without a log / regexp results in a crash #91

Closed bestchai closed 4 years ago

bestchai commented 4 years ago

We should display a friendlier error message (e.g., "You must enter a log before you can parse the log." and "Please specify a regular expression for parsing the log.") when the user presses "Parse Log" without having entered a log or regexp.

[Issue created by brunyuriy: 2016-01-22] [Last updated on bitbucket: 2016-01-28]

[Comment created by ohmann: 2016-01-28] Merged in: 8c90a3c225df

[Comment created by stanleyrya: 2016-01-28] I have a quick fix here in 28159ef21b25, however all it does it display an alert. Do we want to go into more detail and highlight the input box(es) that aren't filled in when the user presses parse?