Skip to content

Conversation

@rlr
Copy link
Contributor

@rlr rlr commented Apr 6, 2012

Turns out we can add STATIC_URL and the collectstatic command in django 1.3.x. If this runs properly, we will be able to get to the admin media files at https://support-dev.allizom.org/static/admin/*. The admin site itself will still use the ADMIN_MEDIA_PREFIX until we update to django 1.4, but this gets us closer to being ready.

r?

* Leaving ADMIN_MEDIA_PREFIX until we update to django 1.4.
@willkg
Copy link
Member

willkg commented Apr 9, 2012

If we're collecting the static files, then we could switch the ADMIN_MEDIA_PREFIX to '/static/admin', right?

@rlr
Copy link
Contributor Author

rlr commented Apr 9, 2012

Yes, but we can't change that until we land django 1.4. For now everything will still be served out of /admin-media/ until /static/ is setup on stage and prod. But we can test the static setup on -dev by hitting the admin media urls directly. Make sense?

@rlr
Copy link
Contributor Author

rlr commented Apr 9, 2012

Actually, the dependency is having /static/ set up on stage and prod, not landing django 1.4.

@rlr
Copy link
Contributor Author

rlr commented Apr 10, 2012

Added --noinput ^

@willkg
Copy link
Member

willkg commented Apr 10, 2012

When I run:

./manage.py collectstatic --noinput

with this branch, it doesn't do anything. It doesn't copy any files. Is it supposed to do something?

@willkg
Copy link
Member

willkg commented Apr 10, 2012

My fault--nevermind. I thought it would always copy over files, but it looks like it only copies over files if they're different or weren't there previously.

We've got the static/ directory checked into git. That might be a complication going forward.

@willkg
Copy link
Member

willkg commented Apr 10, 2012

Scratch the static/ directory in git thing. I misread output from git.

You mentioned adding static to .gitignore---that's a good idea. After that r+.

@rlr
Copy link
Contributor Author

rlr commented Apr 10, 2012

1bcb4b6

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.

2 participants