search

bennylope / django-organizations

:couple: Multi-user accounts for Django projects
http://django-organizations.readthedocs.org/
BSD 2-Clause "Simplified" License
1.31k stars 212 forks source link

fix: include django-extensions as required installations #256

Closed simkimsia closed 1 year ago

simkimsia commented 1 year ago

Because autoslugfield requires django-extensions as indicated here https://github.com/bennylope/django-organizations/blob/2dd044956cf03dad9c8c4954ba325d2769fe3643/docs/getting_started.rst#auto-slug-field

and the minimum version is taken from https://github.com/bennylope/django-organizations/blob/2dd044956cf03dad9c8c4954ba325d2769fe3643/requirements-test.txt#L10

bennylope commented 1 year ago

I'm not overly fond of including this by default, since you can use the package as a "plain" library without needing to install the Django app (thus not needing this dependency).

However the alternatives are (1) status quo or (2) declare named optional dependencies in the setup files... this is probably the simplest way forward. Thank you!