On 18 September 2013 13:13, Martin Grigorov <[email protected]> wrote:

> On Wed, Sep 18, 2013 at 1:28 PM, Martin Grigorov <[email protected]
> >wrote:
>
> > I was able to run quickstart_wicket_restful_jdo webapp successfully.
> >
> > I see Jetty 6.x is used. I'll update it to 8.x.
> >
>
> I've updated it to Jetty 7.6.13.v20130916 because I've noticed that there
> are dependencies to geronimo-servlet-2.5 and jsp-2.1.
> Jetty 8.x requires Servlet API 3.x and maybe there are ISIS users with
> older web servers.
>

given how young Isis is, that's probably unlikely.  But I agree there's no
need to go all the way to Servlet API 3.x at this time.



>
> Why there is a dependency to JSP-2.1 ? I hope ISIS doesn't use JSPs.
>
>
It doesn't.  Or rather, it shouldn't.  Or rather, the Wicket viewer
doesn't.  Would be curious to see why that's added.  If you can't remove it
easily, I'll help.



>
> > Additionally I'll add Start.java (as in Wicket's quickstart) because it
> > allows to develop/debug much faster. Especially when you use DCEVM (
> >
> http://blog.jetbrains.com/idea/2013/07/get-true-hot-swap-in-java-with-dcevm-and-intellij-idea/
> > ).
> >
>

ok


> > At the moment ISIS-537 branch build fails with some issue related to
> > having several versions of ANTLR. We will have to fix it.
>

this might be an interaction with JDO/DN; have seen something related to
this before.



> >
> > Also we (me and Michael) prefer to use Twitter Bootstrap 3, so we will
> > have to use Wicket Bootstrap 0.8.5-SNAPSHOT<
> https://github.com/l0rdn1kk0n/wicket-bootstrap/tree/bootstrap3> build
> > from 'bootstrap3' branch. At the moment these snapshots are not deployed
> > anywhere so every developer will have to clone Wicket Bootstrap project
> > locally and install it (as simple as "mvn clean install")
> >
>

ok



>
> Twitter Bootstrap 3 doesn't support IE8. Is that a problem for ISIS ?
>

Nope, that's fine.

Cheers
Dan

Reply via email to