Several cleanup patches

littlepea / django-editor

Allows pluggable WYSIWYG editors in django admin without hard dependencies

3 stars 2 forks source link

Several cleanup patches #2

Closed dokterbob closed 11 years ago

dokterbob commented 11 years ago

Some cleanup patches during work on implementing configurable editors (see upcoming patch).

Most of this should be pretty basic, except for the South thingy. Turns out django-setuptes barfs when South is installed but not in INSTALLED_APPS.

littlepea commented 11 years ago

Hi,

The code looks fine, but tests/CI fail though...

Could you check if any of the file you removed affect the test runner?

dokterbob commented 11 years ago

Yeah I noticed. Perhaps just ignore this patch and go straight for #3, which passes just fine.

littlepea commented 11 years ago

Sounds good! I'll get back to you on #3