citusdata / django-multitenant

Python/Django support for distributed multi-tenant databases like Postgres+Citus
MIT License
730 stars 118 forks source link

regression with citus distribution migrations being in a different app than the model's app #123

Closed maziar-dandc closed 2 years ago

maziar-dandc commented 2 years ago

Regression happened in: https://github.com/citusdata/django-multitenant/commit/beffbf2025664d4f5f56eb2159e2ac7c2f1af545 Bug fix: https://github.com/Digital-and-Code-UK-Ltd/django-multitenant/commit/30e937d9e5b5158969b57f579a0a9045e8921ffc

Expected behavior: models being distributed properly Actual behavior: models were being distributed as reference tables because they weren't found in get_models() of migration app_label

image

JelteF commented 2 years ago

Hmm, that does not sound good. Could you make a pull request with your changes?