Cool, I've added error field in red like this:
<div id="feedbackErrorMessage" style="color: red;"></div>
And it now warns users (and me) if something is wrong and text hasn't
actually been checked.
Can I also suggest we make 500 message more user-friendly? E.g. with
something like this
diff --git
a/languagetool-server/src/main/java/org/languagetool/server/LanguageToolHttpHandler.java
b/languagetool-server/src/main/java/org/languagetool/server/LanguageToolHttpHandler
.java
index ac6b611..2f16116 100644
---
a/languagetool-server/src/main/java/org/languagetool/server/LanguageToolHttpHandler.java
+++
b/languagetool-server/src/main/java/org/languagetool/server/LanguageToolHttpHandler.java
@@ -227,7 +227,7 @@ class LanguageToolHttpHandler implements HttpHandler {
response = "Checking took longer than " +
maxCheckTimeMillis/1000 + " seconds, which is this server's limit. " +
"Please make sure you have selected the proper
language or consider submitting a shorter text.";
} else {
- response = Tools.getFullStackTrace(e);
+ response = "Internal Error. Please contact site
administrator"; //Tools.getFullStackTrace(e);
errorCode = HttpURLConnection.HTTP_INTERNAL_ERROR;
}
logError(text, remoteAddress, e, errorCode);
2016-04-20 11:03 GMT-04:00 Daniel Naber <daniel.na...@languagetool.org>:
> On 2016-04-20 16:44, Andriy Rysin wrote:
>
>> 4) if I submit a check via the web-page (directly to API, no proxy)
>> there's no messages and text shows up as no errors
>
> I see now - please add <div id="feedbackErrorMessage"></div> to your
> page, it should display the error.
>
> Regards
> Daniel
>
>
> ------------------------------------------------------------------------------
> Find and fix application performance issues faster with Applications Manager
> Applications Manager provides deep performance insights into multiple tiers of
> your business applications. It resolves application problems quickly and
> reduces your MTTR. Get your free trial!
> https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
> _______________________________________________
> Languagetool-devel mailing list
> Languagetool-devel@lists.sourceforge.net
> https://lists.sourceforge.net/lists/listinfo/languagetool-devel
------------------------------------------------------------------------------
Find and fix application performance issues faster with Applications Manager
Applications Manager provides deep performance insights into multiple tiers of
your business applications. It resolves application problems quickly and
reduces your MTTR. Get your free trial!
https://ad.doubleclick.net/ddm/clk/302982198;130105516;z
_______________________________________________
Languagetool-devel mailing list
Languagetool-devel@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/languagetool-devel
