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.

Sign up for GitHub

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

Jump to bottom

SmartModelAdmin is only suitable for registered Admin classes #11

Closed
sveetch opened this issue Apr 7, 2023 · 1 comment
Closed

SmartModelAdmin is only suitable for registered Admin classes #11

sveetch opened this issue Apr 7, 2023 · 1 comment
Labels
enhancement New feature or request

Comments

@sveetch
Copy link
Owner

sveetch commented Apr 7, 2023 •
edited
Loading

SmartModelAdmin can not work with admin class that are not registered to the admin since it inherits from base class Admin.

This results to error when using it with an inline class admin, this can be resolved with using directly SmartModelAdmin.formfield_overrides instead of admin class inheriting from SmartModelAdmin.

However it may be more useful to have something like a mixin or something else we can use in inline class.
@sveetch sveetch added the enhancement New feature or request label Apr 7, 2023
@sveetch sveetch mentioned this issue May 18, 2023
Closed
@sveetch
Copy link
Owner Author

sveetch commented May 19, 2023

Done in 0.3.0-pre.5 with new mixin SmartAdminMixin to inherits instead of SmartModelAdmin

@sveetch sveetch closed this as completed May 19, 2023
sveetch added a commit that referenced this issue May 19, 2023
@sveetch
[NEW] Added 'SmartAdminMixin' to use instead of 'SmartModelAdmin', close
fb48b2e 
 #11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@sveetch