[GitHub] struts pull request: Upgrade tiles plugin

Github user lukaszlenart closed the pull request at: https://github.com/apache/struts/pull/59 --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this feature enabled and wishes so, or if the feature

Re: [GitHub] struts pull request: Upgrade tiles plugin

> > Is there a snapshot that can be used to test it? > I'm using this guide to pull the PR to a local branch and build it: http://struts.apache.org/submitting-patches.html#how-to-merge-pull-requests Regards, Christoph > 2015-11-27 10:18 GMT-05:00 cnenning : >

[GitHub] struts pull request: Upgrade tiles plugin

Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160269259 > BTW what is accessible through `OGNL:` expression? I didn't test that but it should be the whole stack --- If your project is set up for it, you can

[GitHub] struts pull request: Upgrade tiles plugin

Github user aleksandr-m commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160195199 Tested with own app and it is running fine. Looking forward to see the same approach with `StrutsTilesContainerFactory` in tiles3 plugin. BTW what is

Re: [GitHub] struts pull request: Upgrade tiles plugin

Is there a snapshot that can be used to test it? 2015-11-27 10:18 GMT-05:00 cnenning : > Github user cnenning commented on the pull request: > > https://github.com/apache/struts/pull/59#issuecomment-160158437 > > yay, it runs ! :smiley: > > > --- > If your project is

[GitHub] struts pull request: Upgrade tiles plugin

Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160149229 I have the same issue, no idea what's wrong but will try to figure it out. Regarding Tiles 3, it looks very similar to Tiles 2 and I'm going to work on them with

[GitHub] struts pull request: Upgrade tiles plugin

Github user cnenning commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160149460 I hope that code can be shared between both tiles plugins. What do you think of creating a strust-tiles-commons.jar or somthing the like? --- If your project is set up

[GitHub] struts pull request: Upgrade tiles plugin

Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160149850 Will see :) --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have this

[GitHub] struts pull request: Upgrade tiles plugin

Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160157282 @cnenning issue with `javax.el` resolved --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your

[GitHub] struts pull request: Upgrade tiles plugin

Github user cnenning commented on the pull request: https://github.com/apache/struts/pull/59#issuecomment-160158437 yay, it runs ! :smiley: --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as well. If your project does not have