-
Notifications
You must be signed in to change notification settings - Fork 56
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
Implement relationships & tweaks to improve Django admin #74
base: rc/0.2.1
Are you sure you want to change the base?
Conversation
…re cleanup the choices option
Any plans to accept this pull request? |
I am looking at it now @EvdH0 @werneckpaiva. As they say, better late than never. Even when very mega insanely late ? |
neomodel 5.3.0 moved some internals
Fixed things ; the problem was actually that the modifications introduced indirect calls to the prefetch mechanism, which was not implemented in django-neomodel. |
Tests now pass ; but if you open the admin interface and try to list / add some Book / Shelf / Author (so the classes that are modified by this), then it fails. |
The pull request includes a proposal to add functionality to add or remove relationships between nodes. Many-to-Many relationships show up as a
multiple select
elements (and the horizontal/vertical filter widgets can be applied in the Django admin). Additionally minor improvements to the Django admin experience are included.Features
pk
property into an alias instead of a reference to the primary-key (because the Django internals do not addresspk
consistently in the same way). This allows general lookup (and for example fixes the delete button in Django admin)Limitations
choices
of the constructor explicitly (ieSTATUS = {'':'Empty','F': 'Female', 'M': 'Male', 'O': 'Other'}
. Need to check if this is the intended behaviour in neomodel