rykener / django-manifest-loader

Simplifies webpack configuration with Django
https://django-manifest-loader.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
104 stars 12 forks source link

Manifest tag doesn't accept variable argument #17

Closed knokit closed 3 years ago

knokit commented 3 years ago

Trying to use a variable filename:

{% with page|add:".css" as file %}
    <script src="{% manifest file %}" />
{% endwith %}

Results in TemplateSyntaxError: 'manifest' tag's argument should be in quotes.

rykener commented 3 years ago

Hi @knokit thank's for the report. Unfortunately, the fix requires more of a refactor than I anticipated, as the context is not available where the calculations were previously being done. I'm hoping to have something pushed out in the next couple of days.

rykener commented 3 years ago

@knokit this should be fixed for 0.1.0. Thanks again for the report!