Isn't it a good candidate for an external component (i.e. not bundled in OFBiz and maintained by the OFBiz community)? I think we have already discussed this topic some time ago and we concluded that I also cursorily reviewed the patch and I have seen a series of issues (e.g. hardcoded components name in framework code, hardcoded "localhost", bad formatting, misleading variable names like "component" rather than "application" etc...).
Jacopo On Nov 14, 2012, at 7:02 PM, Erwan de FERRIERES wrote: > Hi all, > > I'm planning on committing the patch in OFBIZ-4872 which will add webdriver > testing in OFBiz this week-end. > Is that OK for everyone, or are there some issues with this ? > > Thanks, > > > -- > Erwan de FERRIERES
