Adding Django 1.7+ compatible migrations

feincms / feincms-elephantblog

A blog for FeinCMS

feinheit.ch/labs/

BSD 3-Clause "New" or "Revised" License

43 stars 38 forks source link

Adding Django 1.7+ compatible migrations #57

Closed gxx closed 5 years ago

gxx commented 8 years ago

This fixes issues when the user model is overridden. Due to an issue with the Django migration system, all apps without South/Django migrations are run first. Given that the dependency for the custom user model is not yet set up, the migration of feincms-elephantblog will always fail.

gxx commented 8 years ago

I need to fix tests.

matthiask commented 8 years ago

Thanks! Unfortunately, this particular change does not work very well since Elephantblog has a base model which supports extensions, as FeinCMS' page model does. The same comment as in https://feincms-django-cms.readthedocs.io/en/latest/migrations.html applies: FeinCMS and Elephantblog do not come with migrations, instead, you are expected to fill in MIGRATION_MODULES and run makemigrations & migrate yourself.