awesto / django-shop

A Django based shop system
http://www.django-shop.org
BSD 3-Clause "New" or "Revised" License
3.2k stars 1.03k forks source link

Support for Django 1.10 #386

Closed khchine5 closed 7 years ago

khchine5 commented 8 years ago

Hi all, I have tested django-shop with django 1.10 and all tests pass as shown on travis. Could I make a PR to add support for django 1.10 ?

rfleschenberg commented 8 years ago

Hi,

Yes, please go ahead with the PR!

Unfortunately, we must admit that our tests are somewhat of a weak spot right now, so the fact that they pass with Django 1.10 doesn't mean that much :) Maybe this is a good opportunity to improve our test suite: we should write tests for all bugs we encounter on 1.10 that are not catched by the current test suite.

khchine5 commented 8 years ago

Yes, you are right. I'll contribute by adding some useful tests.

rfleschenberg commented 7 years ago

I fixed travis so that it actually runs the tests on Django 1.10 now :)

ericamador commented 7 years ago

django-shop seems to not be compatible with djangocms-cascade v0.12.0. I get the following error in my project, which seems to be due to this line:

Traceback (most recent call last):
  File "manage.py", line 20, in <module>
    execute_from_command_line(sys.argv)
  File "/Users/eric/.virtualenvs/nimbis/lib/python2.7/site-packages/django/core/management/__init__.py", line 353, in execute_from_command_line
    utility.execute()
  File "/Users/eric/.virtualenvs/nimbis/lib/python2.7/site-packages/django/core/management/__init__.py", line 308, in execute
    settings.configure()
  File "/Users/eric/.virtualenvs/nimbis/lib/python2.7/site-packages/django/conf/__init__.py", line 65, in configure
    raise RuntimeError('Settings already configured.')
RuntimeError: Settings already configured.

This is a blocker for Django 1.10 support, and Django 1.9 only has extended support for a couple of more weeks before it is EOL.

jrief commented 7 years ago

@amadornimbis Django-SHOP 0.10 run with Django-1.10.x and django-CMS-3.4.2