-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Consider removal of Rails support #21
Comments
My 2 cents, tobi Generally I agree with your argument. Furthermore, I assume that most people would use Liquid by hand in their controllers, not exactly as an ActionView Template Handler (eg erb or haml) If that's your line of thought then I think you are right. However, since changing that would effectively be a breaking change I believe it should be part of a larger liquid 3.0.0 roadmap. Finally, assuming I got your point right, I think that removing init itself might not be the necessary. Just the handler part, leaving it to require liquid lib only. Best regards, |
yes this is what I have in mind. Great to have your +1. |
You guys at shopify have something in mind for liquid 3.0? :) |
Got liquid 3 party started at: http://groups.google.com/group/liquid-templates/browse_thread/thread/d6c13a75a3768b18 Hope people participate with their inputs ;) |
That seems like a heavy handed approach for what sounds like essentially an education issue. I've found liquid to be a great addition to a few different rails projects, precisely because of it's intended purpose. No, it's not a general purpose replacement for erb, nor does it seem to be presented as one. But just because I have some developer/designer crafted pages in a project doesn't automatically mean I want to prevent end users from creating content. Conversely, having end user created content doesn't necessarily mean I have no use for developer designed templates. There are plenty of template languages available that work with rails, but none of the others that I've run across are designed to be exposed to a wider audience (and survive). Whether or not it plugs into ActionView in precisely the same way as the general purpose templates is just an implementation detail. But losing (or at least significantly complicating) liquid template use in a rails site would be a very sad direction for liquid to go. |
I'm in the same situation as being described in this thread -- using liquid in a "hostile" environment (multi-tenant rails app, where the tenants can change views). The Liquid templates themselves are stored in the DB, and I was investigating whether writing a custom template resolver (to fetch the template from the DB) and a custom template handler (to actually render the liquid template) makes sense. @tobi -- in your post above, you feel having a template liquid handler does not make sense. Would it be possible for you to explain why? It might not be obvious to someone (me, in this case!) who is tackling this problem for the first time. |
Spend the last hour looking at |
is there more in terms of dependencies to rails despite the i think that nowadays it's more common to install gems instead of using rails plugins. so i think that removing the rails stuff would be fine. if anyone want's to have it, you could go for a liquid-rails.gem like it's done with many other template languages. |
Liquid isn't a very good template engine for rails projects. It's optimized to work in a hostile environment where you cannot trust the people who write the templates. This is different from a development environment in which you should trust your designers and developers who create the templates.
I feel that by including a init.rb we encurage a usage of Liquid that is different from it's actual intend.
The text was updated successfully, but these errors were encountered: