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

Already on GitHub? Sign in to your account

Adding "Using an Embedded Model as an abstract base class" to the docs. #92

Open
wants to merge 1 commit into
from

Conversation

Projects
None yet
3 participants

Nolhian commented Nov 30, 2011

In docs / source / topics / embedded-models.rst

Contributor

jonashaag commented Nov 30, 2011

Thanks for the addition. The issue with the class not being a string is a bug in djangotoolbox though and should be fixed there.

@flaper87 flaper87 closed this Feb 7, 2012

@jonashaag jonashaag referenced this pull request in django-nonrel/djangotoolbox Feb 7, 2012

Open

EmbeddedModelField + lazy lookup for abstract models doesn't work #15

@jonashaag jonashaag reopened this Feb 7, 2012

Contributor

jonashaag commented Feb 7, 2012

We should probably add the "you can use abstract models" thing to the docs once the bug is fixed.

Owner

flaper87 commented Feb 7, 2012

I don't see why we should keep this open if it is a djangotoolbox issue. We can just close it and add the doc string once the bug is fixed.

Contributor

jonashaag commented Feb 7, 2012

Yeah we need to remember to add it to the docs somehow :) You can also open another bug somewhere. Just something to remember it later

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment