Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix #488 com STATIC_ROOT padrão. #493

Merged
merged 1 commit into from Jul 30, 2014
Merged

Conversation

@barraponto
Copy link
Contributor

barraponto commented Jul 1, 2014

No description provided.

@santagada
Copy link
Contributor

santagada commented Jul 1, 2014

Esse não é o jeito certo de fazer isso. static_root por padrão deve ficar em branco. É para vc setar ela só quando for usar, e vai depender da configuração do servidor web.

@barraponto
Copy link
Contributor Author

barraponto commented Jul 1, 2014

Se o certo é deixar o STATIC_ROOT em branco, então imagino que o certo deve ser deixar o app django.contrib.staticfiles desativado também, não?

@santagada
Copy link
Contributor

santagada commented Jul 1, 2014

o staticfiles funciona bem com o static_root em branco em desenvolvimento, para deploy vc tem q setar o static_root mas ai depende de onde esta o seu servidor (um caso é servir de um s3), o static_root só é usado por um commando o collectstatic.

@barraponto
Copy link
Contributor Author

barraponto commented Jul 1, 2014

Eu estou dizendo que dando checkout do master, rodando make e depois ./manage.py runserver, o Django retorna erro 500: You're using the staticfiles app without having set the STATIC_ROOT setting to a filesystem path.

@santagada
Copy link
Contributor

santagada commented Jul 3, 2014

@barraponto
Copy link
Contributor Author

barraponto commented Jul 4, 2014

santagada added a commit that referenced this pull request Jul 30, 2014
@santagada santagada merged commit 59aad25 into hacklabr:master Jul 30, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.