Test and fix for aggregation issue

citusdata / django-multitenant

Python/Django support for distributed multi-tenant databases like Postgres+Citus

MIT License

731 stars 118 forks source link

Test and fix for aggregation issue #64

Closed ledburyb closed 2 years ago

ledburyb commented 4 years ago

Potential solution for #63

citus-bot commented 4 years ago

Hi @ledburyb, thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement.

citus-bot commented 4 years ago

You did it @ledburyb!

Thank you for signing the Citus Data Contributor License Agreement. We can now accept this contribution and all future contributions from you.

Somebody from our team will proceed with code review.

louiseGrandjonc commented 4 years ago

Hi @ledburyb,

Thank you for your contribution! The tests don't pass for several versions. Could you use tox to test with all versions and see why the test you added is failing? Right now it only passes for python 2.7, django 11.

Thanks again