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

Fix to issue #700; should eliminate double-escaping problem. #701

Merged
merged 1 commit into from
Mar 17, 2016

Conversation

legutierr
Copy link

Should fix #700

@carljm
Copy link
Contributor

carljm commented Mar 17, 2016

Looks great, thanks! I'll merge tomorrow.

@carljm
Copy link
Contributor

carljm commented Mar 17, 2016

Looks good to me; merging. Thanks!

carljm added a commit that referenced this pull request Mar 17, 2016
Fix to issue #700; should eliminate double-escaping problem.
@carljm carljm merged commit 983df68 into mozilla:master Mar 17, 2016
carljm added a commit that referenced this pull request Mar 17, 2016
Fix to issue #700; should eliminate double-escaping problem.
carljm added a commit that referenced this pull request Mar 17, 2016
* 2.x:
  Bump version to 2.5.0-dev.2.
  Update version in Changelog, too.
  Release 2.4.1.
  Backport not escaping SafeString; it's a bug.
  Update changelog.
  Merge pull request #701 from legutierr/master
  Merge pull request #623 from atian25/filters-escape
  Throw an error if a block is defined multiple times. Refs #696.
  Update changelog.
  Grammar fixes.
  Merge pull request #691 from dkebler/file-ext
  Update changelog.
  Merge branch 't576'
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

Successfully merging this pull request may close these issues.

Double escaping should be eliminated.
2 participants