On 6 Jul 2017, at 19:09, deshengchuan
<chuandesheng54(a)gmail.com> wrote:
  Sounds good for now. How do you configure if a
check is a warning or an
 error?  
 I'm using a primitive way now actually. I just retrieve the type of
 validator from the error (which is in string), check it against an array of
 validators that are errors. Relevant code snippets are here:
 
https://pastebin.com/4HQukUzU <http://https://pastebin.com/4HQukUzU>  . I
 would appreciate any suggestions to make this code more elegant. 
Ok so it’s hardcoded and thus static ATM.
A simple solution is simply to introduce a configuration API in the CheckerConfiguration
interface:
Map<String, SeverityLevel> getSeverityLevels()
Note that in xwiki.properties it’ll be defined as:
redpen.severityLevels = SuccessiveWord = warning
redpen.severityLevels = DoubleNegative = error
…
  Also, is it possible to try the application right
now? Can I build it and
 install it in my wiki? Is it going to work?  
 I am really sorry for delaying this, but it was working a couple days back
 until I added another bunch of code into it. I will get it working before
 this weekend. 
 
ok thanks
  Not quite clear what you mean. A mockup or
screenshot on the design page
 would be great.  
 I will make mockup and screenshots asap. Meanwhile, my current application
 homepage still remains unchanged, accessible at
 
http://http://design.xwiki.org/xwiki/bin/download/Proposal/RedPenIntegratio…
<http://http://design.xwiki.org/xwiki/bin/download/Proposal/RedPenIntegration/App_home.PNG>
 . 
 
ok I’ve checked it and I don’t really understand why you have an app home page + an Admin
UI page. Aren’t they both about admin configuration of RedPen? Why not a single place?
  You’ve use AWM to create the app? If so you
should instead use the manual
 way as for the FAQ app tutorial here: 
 Yeah I actually used AWM. Will use the manual way to create it.
  I haven’t checked the code yet after my comments
(I think there are still
 some comments I’ve mentioned that don’t have an answer; I remember asking
 you for an example of a setting since I believe a List is not the right
 format).  
 Oops I must have missed them out. Sorry about that! I replied your latest
 github comment; I am unclear towards the nature of the example (do you
 request a code, screenshot etc)
  Just checked the maven build and found several
issues that I’ve fixed.
 Also noticed there’s no functional tests at all.  
 Ok will work on the functional tests asap as well. Thank you for the
 guidance and the fixes, appreciate it very much :) 
 
Thanks
-Vincent