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

All current issues and update for all templating systems #38

Closed
32 tasks done
N0odlez opened this issue Jul 13, 2016 · 15 comments
Closed
32 tasks done

All current issues and update for all templating systems #38

N0odlez opened this issue Jul 13, 2016 · 15 comments

Comments

@N0odlez
Copy link
Contributor

N0odlez commented Jul 13, 2016

Hi Guys,

Just a quick issue for myself really.

I'm going though updating all of the pom's including dependencies and creating unit tests for all of the template engines.

Another thing I'll add if I can get to it is to add Jinjava support as well.

PR will fix: #25 #29 #34 #36

Checklist

Update all dependencies

  • FreeMarker
  • Handlebars
  • Jade
  • Jetbrick This will only be updated to version 2.1.1 as there is currently a issue with 2.1.2 that I have logged
  • JinJava
  • Mustache
  • Pebble
  • ThymeLeaf
  • Velocity
  • Water

Update all readme's

  • FreeMarker
  • Handlebars
  • Jade
  • Jetbrick
  • JinJava
  • Mustache
  • Pebble
  • ThymeLeaf
  • Velocity
  • Water

Create tests and test utilities

  • FreeMarker
  • Handlebars
  • Jade
  • Jetbrick
  • JinJava
  • Mustache
  • Pebble
  • ThymeLeaf
  • Velocity
  • Water

PR is ready! #40

@tipsy
Copy link
Collaborator

tipsy commented Jul 17, 2016

Could you split it into different PRs?

@N0odlez
Copy link
Contributor Author

N0odlez commented Jul 17, 2016

Possibly, I was going to update them all to the latest versions which should just fix a few in one hit. I'll try my best.

@tipsy
Copy link
Collaborator

tipsy commented Jul 17, 2016

You don't have to split every update into separate PRs, just the commit adding Jinjava support.

@N0odlez
Copy link
Contributor Author

N0odlez commented Jul 17, 2016

Created PR #39 to fix #25

@met68
Copy link

met68 commented Aug 2, 2016

Is there only one person maintaining the Spark templates? I keep running into issues with Pebble templates, all to do with the fact that Spark currently only supports Pebble version 1.4.4. Any idea when this might be updated to a more current (ideally THE current) version of Pebble?

P.S. Am I the only person in the world using Spark with Pebble?

@tipsy
Copy link
Collaborator

tipsy commented Aug 2, 2016

Is there only one person maintaining the Spark templates?

Well, it's open source. @perwendel (the founder/main maintainer of Spark) works mostly on the core-framework. I work mainly on new features for the core, and web-stuff. @AzureusNation is doing a great job fixing up all the template engines, and I'll merge his changes once he's done. Maybe you could help him somehow (if he wants help?).

P.S. Am I the only person in the world using Spark with Pebble?

Probably one of very few. Most people use freemarker or velocity, then comes handlebars and mustache. Then maybe jade/thymeleaf? Pebble is a port of Twig (which I think is a PHP template engine?).

If you can't wait, it's pretty easy to use Pebble directly in your project, you don't really need to use Spark's wrapper.

@met68
Copy link

met68 commented Aug 2, 2016

Thanks for your quick reply and yes, I will probably have to use Pebble directly. I would be glad to help, but, alas, I am just getting my feet wet with Java and so would be of little use, I fear. My background is as a relative novice PHP guy and yes, Twig is a PHP templating engine.

@tipsy
Copy link
Collaborator

tipsy commented Aug 2, 2016

If your goal is to get into Java web development with server side rendering, I recommend trying out freemarker. It's very mature, has good IDE support and a lot of powerful features.

@N0odlez
Copy link
Contributor Author

N0odlez commented Aug 3, 2016

@tipsy @met68 I will finish this Sunday at the latest.

@met68
Copy link

met68 commented Aug 6, 2016

@tipsy Thanks. I have taken at cursory glance at Velocity and Freemarker, but feel Pebble (and Twig) are more powerful, especially with things like filters, which I tend to use quite a bit. I have tried to copy-paste the PebbleTemplateEngine class here into my project and update the pom dependency to Pebble 2.2.2 and while this did work, kind of, it did somehow screw-up the charset (I am building a multi-language website which includes German and all the Umlauts broke). I am now back to working with the existing spark-template-pebble 2.3 and using one or two workarounds, until @AzureusNation will hopefully get around to updating spark-template-pebble. As I said, I really appreciate the time you guys dedicate to this!

@N0odlez
Copy link
Contributor Author

N0odlez commented Aug 6, 2016

@met68 I will be pushing tomorrow for sure. Just also creating a PR for something else but will now be working on the templates.

@N0odlez
Copy link
Contributor Author

N0odlez commented Aug 6, 2016

Hmm. I don't know whether to push and create the PR with the current docs or strip them out and update them later... thoughts @tipsy

@tipsy
Copy link
Collaborator

tipsy commented Aug 7, 2016

Won't make much difference. Feel free to just make one giant PR if that's easier.

@tipsy
Copy link
Collaborator

tipsy commented Oct 12, 2016

@AzureusNation did you give up on this?

@tipsy
Copy link
Collaborator

tipsy commented Nov 5, 2016

Took over and fixed most of these

@tipsy tipsy closed this as completed Nov 5, 2016
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

No branches or pull requests

3 participants