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

Add a check to make sure ClusterableModel is used where needed #117

Open
johnraz opened this issue Aug 31, 2019 · 1 comment
Open

Add a check to make sure ClusterableModel is used where needed #117

johnraz opened this issue Aug 31, 2019 · 1 comment

Comments

@johnraz
Copy link

johnraz commented Aug 31, 2019

This is just a proposal but I think it would be nice if we could the Django check framework in order to raise an error when a ParentalKey or a ParentalManyToManyField is used but the model does not inherit from ClusterableModel.

In my case I had to dig and guess on why my many to many field was not saved although everything was executing properly.

Just a suggestion - I don't know how hard that would be to implement.

@gasman
Copy link
Contributor

gasman commented Jul 13, 2020

This already exists for ParentalKey (#35) - sounds like we're missing the equivalent for ParentalManyToManyField.

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

2 participants