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 min and max filters #475

Closed
wants to merge 1 commit into from
Closed

Added min and max filters #475

wants to merge 1 commit into from

Conversation

@snoack
Copy link
Contributor

snoack commented Aug 7, 2015

4 years ago I suggested to add min and max filters, along with some other new filters as part of #66. That pull request has been closed, after some parts of it got implemented independently, even though somebody already complained there about these filters having been left out.

Also I still think adding these particular filters would be useful, as I've seen quite some projects adding custom filters for these purpose. So it seems to be a quite common use case. Also having them built into jinja2, allows us to support the optional attribute and case_sensitive options/semantics used by other built-in filters.

@snoack snoack mentioned this pull request Aug 7, 2015
@snoack snoack force-pushed the snoack:minmax-filters branch from 4123211 to 908db6c Aug 10, 2015
@snoack snoack force-pushed the snoack:minmax-filters branch from 908db6c to 309b10c Aug 10, 2015
@davidism davidism mentioned this pull request Jul 6, 2017
@davidism
Copy link
Member

davidism commented Jul 6, 2017

Continued in #737

@davidism davidism closed this Jul 6, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.