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

should admin integration ignore fields with editable=False by default ? #982

Open
olivierdalang opened this issue Jan 21, 2025 · 0 comments

Comments

@olivierdalang
Copy link

Hi ! Thanks for this awesome package !!

I'm testing it out with a model where I have a fairly large json field to attach some technical information to my instances. This field is declared with editable=False, as it's not meant to be edited through forms. However, django-reversion still serializes it, making the revisions table huge.

IMO, with the default admin integration, it would make sense to exclude fields with editable=False. For specific cases, it's better to manually register the models anyways.

Since this wouldn't be backwards compatible, maybe this behaviour could be behind a setting ?

Let me know what you think. I may be able to contribute this feature if you think it's a good addition.

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

No branches or pull requests

1 participant