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.

Sign up for GitHub

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

Jump to bottom

removed need for any |safe filters #273

Merged
merged 2 commits into from
Sep 9, 2018
Merged

removed need for any |safe filters #273

merged 2 commits into from
Sep 9, 2018

Conversation

josephmancuso
Copy link
Member

This PR removed the need to add the safe filter like {{ csrf_field|safe }} since we just needed to mark the input code with the jinja2.Markup class.

@josephmancuso josephmancuso added beta-2 Labels the issue with which beta version this issue applies to next major Issue scheduled for the next major release labels Sep 3, 2018
@coveralls
Copy link

coveralls commented Sep 3, 2018 •
edited
Loading

Coverage Status

Coverage increased (+2.08%) to 95.734% when pulling 4790b8e on remove-safe-filter into dab5f03 on develop.

@josephmancuso josephmancuso mentioned this pull request Sep 3, 2018
Closed
@josephmancuso josephmancuso merged commit 7986c7d into develop Sep 9, 2018
@josephmancuso josephmancuso deleted the remove-safe-filter branch September 19, 2018 03:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@aisola aisola aisola approved these changes

Assignees
No one assigned
Labels
beta-2 Labels the issue with which beta version this issue applies to next major Issue scheduled for the next major release
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@josephmancuso @coveralls @aisola