search

dimagi / django-cte

Common Table Expressions (CTE) for Django
Other
334 stars 46 forks source link

Hack around missing QJoin.join_field #18

Closed millerdev closed 4 years ago

millerdev commented 4 years ago

Fix regression introduced in Django 3.0.5

Closes #17

dannyroberts commented 4 years ago

✅

warmwaffles commented 4 years ago

@millerdev can you conceive a test that demonstrates the issue noted here? https://github.com/dimagi/django-cte/pull/18/files#diff-143efd239edf672e1ae841f67e79415bR66-R68

millerdev commented 3 years ago

@pgammans I'd love to get your recent contributions into a new release, but I have not had time to fix the hack in this PR. Do you have any input on how to make this more robust and less hacky? See also discussions on https://github.com/dimagi/django-cte/issues/17 and https://github.com/dimagi/django-cte/issues/20.