Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Migrations for EXTRA_MODEL_FIELDS need documentation #1150

Closed
jMyles opened this issue Nov 9, 2014 · 4 comments
Closed

Migrations for EXTRA_MODEL_FIELDS need documentation #1150

jMyles opened this issue Nov 9, 2014 · 4 comments
Labels

Comments

@jMyles
Copy link

jMyles commented Nov 9, 2014

The documentation (https://github.com/stephenmcd/mezzanine/blob/master/docs/model-customization.rst#field-injection-caveats) for this feature doesn't explicitly explain how to do this. Furthermore, since, starting with Django 1.7, migrations need to live in the app which contains the model in question, the process is quite different and non-trivial.

Take a look at this writeup on the process in 1.7 - if there are no objections, I can modify this to fit into the docs.

@stephenmcd
Copy link
Owner

I'd be happy to include a very brief description - no more than a paragraph or two. Really don't want to litter the docs with discussions of workarounds for a misfeature like this. Eventually I hope it can be removed with use of swappable models.

@stephenmcd stephenmcd added the docs label Nov 9, 2014
@jMyles
Copy link
Author

jMyles commented Nov 9, 2014

Ahh, so you regard EXTRA_MODEL_FIELDS as a "misfeature?" Is there a place
I can read the discussion that led to this state?

As I said in my blog post, I will continue to look into swappable models.

On Sun, Nov 9, 2014 at 2:32 AM, Stephen McDonald notifications@github.com
wrote:

I'd be happy to include a very brief description - no more than a
paragraph or two. Really don't want to litter the docs with discussions of
workarounds for a misfeature like this. Eventually I hope it can be removed
with use of swappable models.


Reply to this email directly or view it on GitHub
#1150 (comment)
.

@AlexHill
Copy link
Collaborator

Hi Steve,

I'm interested to hear what you'd like swappable models in Mezzanine to
look like. I think it's a great idea, particularly for Cartridge.

Alex

On Sunday, 9 November 2014, Stephen McDonald notifications@github.com
wrote:

I'd be happy to include a very brief description - no more than a
paragraph or two. Really don't want to litter the docs with discussions of
workarounds for a misfeature like this. Eventually I hope it can be removed
with use of swappable models.


Reply to this email directly or view it on GitHub
#1150 (comment)
.

@stephenmcd
Copy link
Owner

I've created a new issue #1160 with initial thoughts.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants