-
Notifications
You must be signed in to change notification settings - Fork 20
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
Implement all filters #3
Comments
Going to try helping where I can :), already did my first PR for downcase. |
Thanks for your contribution 🍰 |
As for the |
Using the [] operator can be a solution, but it means that the map filter will be restricted to hash :/ |
Maybe this could be a reason for them to support this feature in the language? I ran into a similar issue when trying to create php style dynamic controller/actions as well and was forced to use a single method as the solution :/ I almost had a breakthrough though, basically i found out that constant arrays can be used dynamically in |
actually now looking at the liquid source, it looks like they only really support Enumerable's w/ the map option that responds to Guess like you said, that'll have to do for now. My current solution: exts@c03a643 will push later once I add more filters. |
Filters
The text was updated successfully, but these errors were encountered: