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

Missing requirements in the "Take a peek" section #180

Closed
apollo13 opened this issue May 13, 2012 · 3 comments
Closed

Missing requirements in the "Take a peek" section #180

apollo13 opened this issue May 13, 2012 · 3 comments

Comments

@apollo13
Copy link

./manage.py loaddata countries.json fixtures/pages.json

in http://tangentlabs.github.com/django-oscar/#takeapeek requires the user to install debug_toolbar and django_extensions -- this is not installed via python setup.py develop!

@AndrewIngram
Copy link
Contributor

What's the best way to handle development dependencies? We don't want to install debug_toolbar and django_extensions for all users of Oscar, just those who plan to hack on it.

@apollo13
Copy link
Author

Well given that the other commands all use --settings=settings_simple your loaddata probably should use it too... Another option would be to require the user a pip install -r development_packages etc.

@codeinthehole
Copy link
Contributor

My bad - forgot to specify the simpler settings file (which omits all the testing dependencies). Now fixed.

There is a requirements.txt file with the testing dependencies in it. However, I had thought it unnecessary to make users install these packages as they don't need them to try out the sandbox site - hence the settings_simple.py file.

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

No branches or pull requests

3 participants