search

joestump / django-ajax

A simple framework for creating AJAX endpoints in Django.
http://github.com/joestump/django-ajax
BSD 3-Clause "New" or "Revised" License
260 stars 53 forks source link

django-taggit requirement #30

Closed ephess closed 12 years ago

ephess commented 12 years ago

django-taggit needs to be specified in setup.py as a requirement:

Error: No module named taggit.utils
joestump commented 12 years ago

Good call. I think I'll probably make this throw an exception when you try to use the tagging feature as "not implemented". Then it'd degrade gracefully and not require taggit. How's that sound?

ephess commented 12 years ago

Sounds good, I've no need for the tagging functionality at this stage as it's my first time using the project so suits me fine.

Perhaps an optional requirements section is in order? The extra_requires option seems like it would be a good fit here?

http://packages.python.org/distribute/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies

Cheers for the quick response.

joestump commented 12 years ago

I'm going to do both, I think.

On Wednesday, August 22, 2012 at 3:13 PM, Jordan Hagan wrote:

Sounds good, I've no need for the tagging functionality at this stage as it's my first time using the project so suits me fine. Perhaps an optional requirements section is in order? The extra_requires option seems like it would be a good fit here? http://packages.python.org/distribute/setuptools.html#declaring-extras-optional-features-with-their-own-dependencies Cheers for the quick response.

— Reply to this email directly or view it on GitHub (https://github.com/joestump/django-ajax/issues/30#issuecomment-7953869).