Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Added EL OGNL Freemarker Tiles Factory #7

Open
wants to merge 1 commit into
base: TILES_3_0_X
Choose a base branch
from
Open

Added EL OGNL Freemarker Tiles Factory #7

wants to merge 1 commit into from

Conversation

dragos-onac
Copy link

It only supports one tiles.xml (/WEB-INF/tiles.xml)file
It has the minimum set of dependencies enabling one to run EL and OGNL tiles

It only supports one tiles.xml (/WEB-INF/tiles.xml)file
It has the minimum set of dependencies enabling one to run EL and OGNL tiles
@michaelsembwever
Copy link
Member

Thanks for this @dragos-onac

Since this is more an example of how to set up (configure) Tiles to work with EL and OGNL I think this would be better suited to live under a examples section…

@michaelsembwever
Copy link
Member

we have https://svn.apache.org/viewvc/tiles/examples/trunk/ which is likely very outdated now and never got sync to git.

so i think that a new submodules tiles-examples can be created. then create a example-el-ognl submodule again under that…

wdyt @nlebas ?

@dragos-onac
Copy link
Author

Hi Michael,
Sorry it took so long replying, I also tried uplifting the former tiles-example... That proved a bit too much for now :)
Will submit the porposed changes in a new module in tiles/examples/example-el-ognl .

Thank you!
Dragos

@michaelsembwever
Copy link
Member

much appreciated @dragos-onac

@dragos-onac
Copy link
Author

dragos-onac commented Jul 19, 2016

Done in a new pull, here #10

It was easier to sync with the latest version

Dragos

From: mck [mailto:notifications@github.com]
Sent: Monday, July 18, 2016 11:25 PM
To: apache/tiles tiles@noreply.github.com
Cc: Dragos Onac dragos.onac@liss.co.uk; Mention mention@noreply.github.com
Subject: Re: [apache/tiles] Added EL OGNL Freemarker Tiles Factory (#7)

much appreciated @dragos-onachttps://github.com/dragos-onac

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants