Nice branch! Small, focused and easy to follow :) It's good to land with a couple of nits to fix as below:
[1]
174 -{{ preseed }} 175 +{{ preseed|force_escape }}
Was there a reason to use force_escape here? Django's docs say to use just "escape" unless you want immediate escaping, which I don't think is the case?
https://docs.djangoproject.com/en/dev/ref/templates/builtins/#std:templatefilter-escape
[2]
The tests for escaping could be improved if they did a more explicit test, such as looking for <mytag> when the preseed is <mytag>.
« Back to merge proposal
Nice branch! Small, focused and easy to follow :) It's good to land with a couple of nits to fix as below:
[1]
174 -{{ preseed }} force_escape }}
175 +{{ preseed|
Was there a reason to use force_escape here? Django's docs say to use just "escape" unless you want immediate escaping, which I don't think is the case?
https:/ /docs.djangopro ject.com/ en/dev/ ref/templates/ builtins/ #std:templatefi lter-escape
[2]
The tests for escaping could be improved if they did a more explicit test, such as looking for <mytag> when the preseed is <mytag>.