Skip to content

Update django_simple_bulma.py#36

Closed
krlan wants to merge 1 commit into
lemonsaurus:masterfrom
krlan:patch-1
Closed

Update django_simple_bulma.py#36
krlan wants to merge 1 commit into
lemonsaurus:masterfrom
krlan:patch-1

Conversation

@krlan
Copy link
Copy Markdown

@krlan krlan commented Dec 10, 2019

No static templatetag in newer versions

No static templatetag in newer versions
@MarkKoz MarkKoz requested a review from lemonsaurus December 10, 2019 06:52
@lemonsaurus
Copy link
Copy Markdown
Owner

Hey @krlan, thanks for the PR! Do you know if this breaks compatibility with older versions of Django?

Looks like the build is currently failing with this error:

django.core.exceptions.ImproperlyConfigured: You're using the staticfiles app without having set the required STATIC_URL setting.

Could you have a look at this and see if you can get the build passing?

@MarkKoz MarkKoz mentioned this pull request Dec 19, 2019
Copy link
Copy Markdown
Contributor

@MarkKoz MarkKoz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Per the discussion #37 (comment), it's recommend and more convenient to use django.templatetags.static.static() instead.

@lemonsaurus
Copy link
Copy Markdown
Owner

We merged #38 instead, which seems to have the prefered solution.

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.

3 participants