Skip to content

Change 'staticfiles' to 'static' with new Django version - #95

Merged
edmorley merged 1 commit into
mainfrom
fix_deprecation_of_static_files
Mar 18, 2021
Merged

Change 'staticfiles' to 'static' with new Django version#95
edmorley merged 1 commit into
mainfrom
fix_deprecation_of_static_files

Conversation

@ericc572

Copy link
Copy Markdown

{% load staticfiles %} and {% load admin_static %} were deprecated in Django 2.1, and removed in Django 3.0. see last bullet here

Tried to follow the getting started guide and ran into this issue when going to the '/db' route. Would create a more consistent experience for all subsequent users for this guide in the future!

@ericc572
ericc572 requested a review from CaseyFaist March 28, 2020 00:08
@edmorley
edmorley changed the base branch from master to main September 3, 2020 19:55

@zamirszn zamirszn left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

mistake ,sorry

@mdebono mdebono left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Did the same change for myself and then found out there was this pull request. Thanks and would be nice for others to have this merged.

@sponchcafe

Copy link
Copy Markdown

Hi, I ran into this problem (and fix) today. Will this be merged anytime soon? No pressure, I know people have stuff to do :)

@0xNF

0xNF commented Jan 22, 2021

Copy link
Copy Markdown

An easy bug to fix, but one everyone will encounter. Shouldn't take almost a year to merge this in, come on guys.

@edmorley

Copy link
Copy Markdown
Member

This is in my backlog, but behind higher priority Python buildpack work. Also I first need to set up CI for this repo (likely Heroku CI which would have the added benefit of demonstrating that too).

@marcelloromani

Copy link
Copy Markdown

@edmorley this just needs to be merged, it's a relatively obscure django detail that has nothing to do with heroku itself, and is just a bump in an otherwise flawless tutorial. What's stopping you (or anyone in charge, really) from merging this no-brainer?

@heroku
heroku Bot temporarily deployed to python-getti-fix-deprec-yvzhkt March 18, 2021 13:46 Inactive
@edmorley
edmorley force-pushed the fix_deprecation_of_static_files branch from 797dbd1 to 0b0c0c7 Compare March 18, 2021 13:47
@heroku
heroku Bot temporarily deployed to python-getti-fix-deprec-o84pea March 18, 2021 13:48 Inactive
@edmorley
edmorley merged commit 0571b35 into main Mar 18, 2021
@edmorley
edmorley deleted the fix_deprecation_of_static_files branch March 18, 2021 13:51
@edmorley

Copy link
Copy Markdown
Member

Thank you for the PR! Sorry it's taken so long to review/merge -- I've now set up CI and Review Apps on this repo so it should be faster to validate from now on :-)

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.

8 participants