Skip to content
This repository has been archived by the owner on Jan 18, 2020. It is now read-only.

Replace use of real models with South's orm-based ones #32

Merged
merged 1 commit into from
Oct 16, 2014

Conversation

naegelyd
Copy link
Collaborator

Fix #30 and #31.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 95acc11 on orm-models-in-migrations into 80e65d7 on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling 95acc11 on orm-models-in-migrations into 80e65d7 on master.


orm['avocado.DataField'].objects.get_or_create(
app_name='variants', model_name='variantphenotype',
field_name='hgmd_id', name='HGMD', published=True)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

name and published need to be in the defaults dict for get_or_create since it should not apply to the lookup: https://docs.djangoproject.com/en/1.5/ref/models/querysets/#get-or-create

@bruth
Copy link
Contributor

bruth commented Oct 16, 2014

Since Avocado is now serialized in all of those migrations.. they depend on the most recent version since I see indexable (for example) in there.

@bruth
Copy link
Contributor

bruth commented Oct 16, 2014

The depends_on property should be added pointing to the migration including the indexable field: 0035_auto__add_field_datafield_indexable__add_field_dataconcept_indexable

These changes also include frozen versions of the avocaod, auth, and
contenttypes models to be included in the migration.

Signed-off-by: Don Naegely <naegelyd@gmail.com>
bruth added a commit that referenced this pull request Oct 16, 2014
Replace use of real models with South's orm-based ones
@bruth bruth merged commit f378f84 into master Oct 16, 2014
@bruth bruth deleted the orm-models-in-migrations branch October 16, 2014 18:38
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

south migration complaining about avocado_datafield.indexable
3 participants